[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/misc/MacroRepeatedSideEffectsCheck.cpp b/clang-tools-extra/clang-tidy/misc/MacroRepeatedSideEffectsCheck.cpp
index 24593cba172..e8ad7c57efc 100644
--- a/clang-tools-extra/clang-tidy/misc/MacroRepeatedSideEffectsCheck.cpp
+++ b/clang-tools-extra/clang-tidy/misc/MacroRepeatedSideEffectsCheck.cpp
@@ -65,9 +65,9 @@ void MacroRepeatedPPCallbacks::MacroExpands(const Token &MacroNameTok,
     if (hasSideEffects(ResultArgToks) &&
         countArgumentExpansions(MI, Arg) >= 2) {
       Check.diag(ResultArgToks->getLocation(),
-                 "side effects in the %ordinal0 macro argument '%1' are "
+                 "side effects in the %ordinal0 macro argument %1 are "
                  "repeated in macro expansion")
-          << (ArgNo + 1) << Arg->getName();
+          << (ArgNo + 1) << Arg;
       Check.diag(MI->getDefinitionLoc(), "macro %0 defined here",
                  DiagnosticIDs::Note)
           << MacroNameTok.getIdentifierInfo();