[clang] Enable frame pointer for non-leaf functions on Android (PR #97614)
Sam Elliott via cfe-commits
cfe-commits at lists.llvm.org
Fri Aug 23 06:06:03 PDT 2024
================
@@ -166,7 +155,7 @@ static bool useFramePointerForTargetByDefault(const llvm::opt::ArgList &Args,
static bool useLeafFramePointerForTargetByDefault(const llvm::Triple &Triple) {
if (Triple.isAArch64() || Triple.isPS() || Triple.isVE() ||
- (Triple.isAndroid() && Triple.isRISCV64()))
+ (Triple.isAndroid() && !Triple.isARM()))
----------------
lenary wrote:
Did you mean to miss out Thumb? `isARM` is only true for `arm-*`, `armeb-*` triples, you would need an `isThumb` call to also cover `thumb-*` and `thumbeb-*` triples.
https://github.com/llvm/llvm-project/pull/97614
More information about the cfe-commits
mailing list