[PATCH] D80828: [Clang][A32/T32][Linux] -O2 implies -fomit-frame-pointer

[Stephen Hines via Phabricator via cfe-commits]

srhines added inline comments.


================
Comment at: clang/lib/Driver/ToolChains/Clang.cpp:560
     // Don't use a frame pointer on linux if optimizing for certain targets.
+    case llvm::Triple::arm:
+    case llvm::Triple::armeb:
----------------
For the new targets, we should only be changing the default for non-Android cases. Android targets should still default to `-fno-omit-frame-pointer`. This makes the logic here quite a bit more complex.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D80828/new/

https://reviews.llvm.org/D80828