[clang] [clang] fix uefi target for aarch64 & x86_64 (PR #120632)

via cfe-commits cfe-commits at lists.llvm.org
Sat Dec 21 17:50:21 PST 2024


================
@@ -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:

Merged the clean up PR. @petrhosek's thoughts on DefineMacro over DefineStd is sound in my opinion.  Going to merge the predefine PR as well #111719 which we can revert/reland if we have significant push back and deal with it in a way that it doesn't block this PR.

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


More information about the cfe-commits mailing list