[PATCH] D24845: [clang-tidy] fix for NOLINT after macro expansion

Alexander Kornienko via cfe-commits cfe-commits at lists.llvm.org
Fri Sep 23 19:13:11 PDT 2016


alexfh accepted this revision.
alexfh added a comment.
This revision is now accepted and ready to land.

Awesome, thanks!


================
Comment at: clang-tidy/ClangTidyDiagnosticConsumer.cpp:303
@@ +302,3 @@
+    if (!Loc.isMacroID())
+      break;
+    Loc = SM.getImmediateExpansionRange(Loc).first;
----------------
Just `return false;`?


https://reviews.llvm.org/D24845





More information about the cfe-commits mailing list