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

[Eli Friedman via Phabricator via cfe-commits]

efriedma added a comment.

Please start a thread on cfe-dev about this; most developers don't read cfe-commits, and thinking about it a bit more, I'm not confident omitting frame pointers is really a good default.  I would guess there's existing code which depends on frame pointers to come up with a stack trace, since table-based unwinding is complicated and takes up a lot of space.

Grepping over the in-tree tests, it looks like the "eabi" triples used in practice are "arm-non-eabi", "arm-netbsd-eabi" and variants of "thumbv7m-apple-darwin-eabi" (and variants of those).  Please make sure you aren't changing the behavior of netbsd and "Darwin" targets.


https://reviews.llvm.org/D31972