[clang] Patch series to reapply #118734 and substantially improve it (PR #120534)

Chandler Carruth via cfe-commits cfe-commits at lists.llvm.org
Thu Dec 19 10:07:02 PST 2024


================
@@ -43,7 +43,8 @@ class LLVM_LIBRARY_VISIBILITY XCoreTargetInfo : public TargetInfo {
   void getTargetDefines(const LangOptions &Opts,
                         MacroBuilder &Builder) const override;
 
-  ArrayRef<Builtin::Info> getTargetBuiltins() const override;
+  std::pair<const llvm::StringTable *, ArrayRef<Builtin::Info>>
----------------
chandlerc wrote:

Yeah. I mostly didn't invest in tidying this because I knew it was going to change. ;]

Are you OK with just letting the series clean this up, or would you prefer a type alias in the initial patch?

https://github.com/llvm/llvm-project/pull/120534


More information about the cfe-commits mailing list