-
Notifications
You must be signed in to change notification settings - Fork 13k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Turn off DAGCombine at -O0 #22720
Comments
Here's the list of failing tests that I got: Failing Tests (19): CodeGen/ARM/2010-05-18-LocalAllocCrash.ll crashes in llc: CodeGen/X86/avx-load-store.ll gets a selection failure: |
I don't think this is an entirely reasonable goal. DAGCombiner performs various canonicalizations the targets rely on, and being able to disable it increases the complexity they will need to deal with. There isn't really a way to have separate phases with the current DAG, so these aren't cleanly separated from "real" optimizations. |
Extended Description
DAGCombine should not run at -O0 (see discussion on review D7181).
http://reviews.llvm.org/D7181
However, doing that at the moment causes 19 test failures, including
one crash and one fail-to-select.
The text was updated successfully, but these errors were encountered: