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



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

Reply via email to