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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to