[PowerPC] Moving defineXLCompatMacros() definition

[quinnp]

Summary

[NFC] Moving the definition of the defineXLCompatMacros function from
the header file clang/lib/Basic/Targets/PPC.h to the source file
clang/lib/Basic/Targets/PPC.cpp.

[lei]

LGTM
Thanks.

clang/lib/Basic/Targets/PPC.h

public:
   }
   const char *getFloat128Mangling() const override { return "u9__ieee128"; }
 
   bool hasExtIntType() const override { return true; }
 
   bool isSPRegName(StringRef RegName) const override {
     return RegName.equals("r1") || RegName.equals("x1");
   }
-
-  void defineXLCompatMacros(MacroBuilder &Builder) const {
-    Builder.defineMacro("__popcntb", "__builtin_ppc_popcntb");
-    Builder.defineMacro("__eieio", "__builtin_ppc_eieio");
-    Builder.defineMacro("__iospace_eieio", "__builtin_ppc_iospace_eieio");
-    Builder.defineMacro("__isync", "__builtin_ppc_isync");
-    Builder.defineMacro("__lwsync", "__builtin_ppc_lwsync");
-    Builder.defineMacro("__iospace_lwsync", "__builtin_ppc_iospace_lwsync");
-    Builder.defineMacro("__sync", "__builtin_ppc_sync");
-    Builder.defineMacro("__iospace_sync", "__builtin_ppc_iospace_sync");
-    Builder.defineMacro("__dcbfl", "__builtin_ppc_dcbfl");
-    Builder.defineMacro("__dcbflp", "__builtin_ppc_dcbflp");
-    Builder.defineMacro("__dcbst", "__builtin_ppc_dcbst");
-    Builder.defineMacro("__dcbt", "__builtin_ppc_dcbt");
-    Builder.defineMacro("__dcbtst", "__builtin_ppc_dcbtst");
-    Builder.defineMacro("__dcbz", "__builtin_ppc_dcbz");
-    Builder.defineMacro("__icbt", "__builtin_ppc_icbt");
-  }
 };
 
 class LLVM_LIBRARY_VISIBILITY PPC32TargetInfo : public PPCTargetInfo {
 public:
   PPC32TargetInfo(const llvm::Triple &Triple, const TargetOptions &Opts)
       : PPCTargetInfo(Triple, Opts) {
     if (Triple.isOSAIX())
       resetDataLayout("E-m:a-p:32:32-i64:64-n32");

clang/lib/Basic/Targets/PPC.cpp

for (const auto &Feature : Features) {
     }
     // TODO: Finish this list and add an assert that we've handled them
     // all.
   }
 
   return true;
 }
 
+static void defineXLCompatMacros(MacroBuilder &Builder) {
+  Builder.defineMacro("__popcntb", "__builtin_ppc_popcntb");
+  Builder.defineMacro("__eieio", "__builtin_ppc_eieio");
+  Builder.defineMacro("__iospace_eieio", "__builtin_ppc_iospace_eieio");
+  Builder.defineMacro("__isync", "__builtin_ppc_isync");
+  Builder.defineMacro("__lwsync", "__builtin_ppc_lwsync");
+  Builder.defineMacro("__iospace_lwsync", "__builtin_ppc_iospace_lwsync");
+  Builder.defineMacro("__sync", "__builtin_ppc_sync");
+  Builder.defineMacro("__iospace_sync", "__builtin_ppc_iospace_sync");
+  Builder.defineMacro("__dcbfl", "__builtin_ppc_dcbfl");
+  Builder.defineMacro("__dcbflp", "__builtin_ppc_dcbflp");
+  Builder.defineMacro("__dcbst", "__builtin_ppc_dcbst");
+  Builder.defineMacro("__dcbt", "__builtin_ppc_dcbt");
+  Builder.defineMacro("__dcbtst", "__builtin_ppc_dcbtst");
+  Builder.defineMacro("__dcbz", "__builtin_ppc_dcbz");
+  Builder.defineMacro("__icbt", "__builtin_ppc_icbt");
+}
+
 /// PPCTargetInfo::getTargetDefines - Return a set of the PowerPC-specific
 /// #defines that are not tied to a specific subtarget.
 void PPCTargetInfo::getTargetDefines(const LangOptions &Opts,
                                      MacroBuilder &Builder) const {
 
   defineXLCompatMacros(Builder);
 
   // Target identification.