[PATCH] D31972: Do not force the frame pointer by default for ARM EABI

Eli Friedman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jun 1 12:10:57 PDT 2017


efriedma added inline comments.


================
Comment at: lib/Driver/ToolChains/Clang.cpp:569
 
+  if (Triple.getEnvironment() == llvm::Triple::EABI) {
+    switch (Triple.getArch()) {
----------------
chrib wrote:
> efriedma wrote:
> > Specifically checking for "llvm::Triple::EABI" is suspicious... what are you trying to distinguish?
> I'm targeting the AAPCS for bare toolsets, (we could also test EABIHF by the way)
> 
> I'm not sure about the other ABIs (such as llvm::Triple::OpenBSD) so it is probably conservative and stick to what I can test. Do you think this pertains to more, for instance to AAPCS-LINUX, without breaking anything ?
> 
So... something like isTargetAEABI() in ARMSubtarget.h?

Please clarify the comment, and add a check for EABIHF.


https://reviews.llvm.org/D31972





More information about the cfe-commits mailing list