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

[Stephen Hines via Phabricator via llvm-commits]

srhines added inline comments.


================
Comment at: clang/lib/Driver/ToolChains/Clang.cpp:565
+    case llvm::Triple::thumbeb:
+      if (Triple.getEnvironment() == llvm::Triple::Android)
+        return true;
----------------
Triple.isAndroid() is more clear.


================
Comment at: llvm/docs/ReleaseNotes.rst:94
 
+* Clang now defaults to `-fomit-frame-pointer` when targeting Linux for arm
+  and thumb when optimizations are enabled. Users that were previously not
----------------
"non-Android Linux" is probably going to make things easier to understand here.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80828