================ @@ -788,16 +789,28 @@ class LLVM_LIBRARY_VISIBILITY ZOSTargetInfo : public OSTargetInfo<Target> { // UEFI target template <typename Target> class LLVM_LIBRARY_VISIBILITY UEFITargetInfo : public OSTargetInfo<Target> { + llvm::Triple Triple; + protected: void getOSDefines(const LangOptions &Opts, const llvm::Triple &Triple, - MacroBuilder &Builder) const override {} + MacroBuilder &Builder) const override { + DefineStd(Builder, "uefi", Opts); ---------------- Prabhuk wrote:
I have approvals to land #111719 and #111473 which breaks out some aspects of this large PR. I am happy to either land them first or wait for all those changes to be part of this PR. @RossComputerGuy I am happy to go with whichever way makes landing this patch faster. Let me know. https://github.com/llvm/llvm-project/pull/120632 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits