[clang-tidy] Don't emit google-runtime-references warning for functions defined in macros.

[hokein]

Summary

The macro are usually defined in the common/base headers which are hard
for normal users to modify it.

[gribozavr]

LGTM, although I'd be more comfortable with a whitelist of macros.

clang-tools-extra/test/clang-tidy/google-runtime-references.cpp

156 ↗

(On Diff #216775)

Please add back the newline.

[hokein]

Add the missing back newline.

clang-tools-extra/trunk/clang-tidy/google/NonConstReferences.cpp

 void NonConstReferences::check(const MatchFinder::MatchResult &Result) {
   const auto *Parameter = Result.Nodes.getNodeAs<ParmVarDecl>("param");
   const auto *Function =
       dyn_cast_or_null<FunctionDecl>(Parameter->getParentFunctionOrMethod());
 
   if (Function == nullptr || Function->isImplicit())
     return;
 
+  if (Function->getLocation().isMacroID())
+    return;
+
   if (!Function->isCanonicalDecl())
     return;
 
   if (const auto *Method = dyn_cast<CXXMethodDecl>(Function)) {
     // Don't warn on implementations of an interface using references.
     if (Method->begin_overridden_methods() != Method->end_overridden_methods())
       return;
     // Don't warn on lambdas, as they frequently have to conform to the

clang-tools-extra/trunk/test/clang-tidy/google-runtime-references.cpp

 namespace whitelist {
 class A {};
 class B {};
 void f7(A &);
 void f8(B &);
 }
 void f9(whitelist::A &);
 void f10(whitelist::B &);
+
+#define DEFINE_F(name) void name(int& a)
+
+DEFINE_F(func) {}