[clang] [clang][ARM] disable frame pointers by default for bare metal ARM targets (PR #117140)

Ties Stuij via cfe-commits cfe-commits at lists.llvm.org
Tue Nov 26 03:50:31 PST 2024


================
@@ -151,6 +152,9 @@ static bool useFramePointerForTargetByDefault(const llvm::opt::ArgList &Args,
     }
   }
 
+  if (toolchains::isARMBareMetal(Triple))
----------------
stuij wrote:

Hi @jroelofs, adding the check for Apple shouldn't be necessary. `isARMBareMetal` is trying to stay in it's lane by only returning true if the vendor is `none` and the environment is `eabi` or `eabihf`.

>From grepping the LLVM tests I understand the Apple firmware triple would be something like `arm/thumb-apple-none-macho`, correct? I've added a number of tests to cover this.

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


More information about the cfe-commits mailing list