[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