[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