[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