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

Eric Christopher via cfe-commits cfe-commits at lists.llvm.org
Thu Dec 19 12:50:48 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__");
----------------
echristo wrote:

Personally I prefer DefineStd for this area where possible or we know we're not adding conflicting macros. What do other toolchains produce here?

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


More information about the cfe-commits mailing list