[PATCH] D24173: Explicitly require DominatorTreeAnalysis pass for instsimplify pass.
Chandler Carruth via llvm-commits
llvm-commits at lists.llvm.org
Thu Sep 8 02:37:12 PDT 2016
chandlerc added a subscriber: chandlerc.
chandlerc added a comment.
The change seems fine, but you should add a test that demonstrates the failure. Any IR input that triggers the code path you suggest run with just the inst simplify pass from opt should suffice?
https://reviews.llvm.org/D24173
More information about the llvm-commits
mailing list