hiraditya added inline comments.
================
Comment at: clang/lib/Driver/ToolChains/Clang.cpp:424
+ if (Triple.isAndroid()) {
+ // AArch64 has frame pointers enabled for non-leaf functions.
+ switch (Triple.getArch()) {
----------------
hiraditya wrote:
> enh wrote:
> > (where? is it simpler to just add arm64 to the switch?)
> yeah we can add it for better readability but it would be redundant.
seems like we don't even have a test for aarch64.*android target. I'll add a
testcase.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D150490/new/
https://reviews.llvm.org/D150490
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
