[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