================
@@ -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
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
