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

Joseph Huber via cfe-commits cfe-commits at lists.llvm.org
Thu Dec 19 12:27:38 PST 2024


================
@@ -790,7 +790,9 @@ template <typename Target>
 class LLVM_LIBRARY_VISIBILITY UEFITargetInfo : public OSTargetInfo<Target> {
 protected:
   void getOSDefines(const LangOptions &Opts, const llvm::Triple &Triple,
-                    MacroBuilder &Builder) const override {}
+                    MacroBuilder &Builder) const override {
+    Builder.defineMacro("__UEFI__");
----------------
jhuber6 wrote:

I wonder if this should be consistent with the Unix flavors, I don't know what the UEFI target does or if GCC or another other compiler already has a macro for this.
```suggestion
    DefineStd(Builder, "uefi", Opts);
```

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


More information about the cfe-commits mailing list