[clang] [llvm] [ARM] __ARM_ARCH macro definition fix (PR #81493)
via cfe-commits
cfe-commits at lists.llvm.org
Tue Feb 13 02:36:49 PST 2024
================
@@ -736,9 +737,17 @@ void ARMTargetInfo::getTargetDefines(const LangOptions &Opts,
if (!CPUAttr.empty())
Builder.defineMacro("__ARM_ARCH_" + CPUAttr + "__");
- // ACLE 6.4.1 ARM/Thumb instruction set architecture
- // __ARM_ARCH is defined as an integer value indicating the current ARM ISA
- Builder.defineMacro("__ARM_ARCH", Twine(ArchVersion));
+ // ACLE 5.4.1 ARM/Thumb instruction set architecture
----------------
ostannard wrote:
This macro doesn't actually have anything to do with the ARM/Thumb ISAs, this line of the comment can just be deleted.
https://github.com/llvm/llvm-project/pull/81493
More information about the cfe-commits
mailing list