[clang-tidy] Remove unnecessary getName() on Decls and Types feeding into a DiagnosticBuilder

Going through a string removes some of the smarts of the diagnosic printer
and makes the code more complicated. This change has some cosmetic impact
on the output but that's mostly minor.

llvm-svn: 265680

1 files changed, 2 insertions, 2 deletions

diff --git a/clang-tools-extra/clang-tidy/google/NonConstReferences.cpp b/clang-tools-extra/clang-tidy/google/NonConstReferences.cpp
index cafc908d8a7..5667bd19502 100644
--- a/clang-tools-extra/clang-tidy/google/NonConstReferences.cpp
+++ b/clang-tools-extra/clang-tidy/google/NonConstReferences.cpp
@@ -115,8 +115,8 @@ void NonConstReferences::check(const MatchFinder::MatchResult &Result) {
         << Parameter->getFunctionScopeIndex();
   } else {
     diag(Parameter->getLocation(),
-         "non-const reference parameter '%0', make it const or use a pointer")
-        << Parameter->getName();
+         "non-const reference parameter %0, make it const or use a pointer")
+        << Parameter;
   }
 }