[PATCH] D41362: [AArch64][GlobalISel] Enable GlobalISel at -O0 by default

Quentin Colombet via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 20 15:23:20 PST 2017

qcolombet added a comment.

Sounds like this option doesn’t use the fallback path.
I thought it did.
Something to double check.

Comment at: lib/CodeGen/TargetPassConfig.cpp:141
+             "3 disable abort if global-isel is enabled by default"),
+    cl::init(3));
I would rather have another target hook for the abort case.
Same thing as what we have for GISel enabled (option takes precedence if set, otherwise we look at what the target wants)



More information about the llvm-commits mailing list