================ @@ -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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits