[PATCH] D32476: [EarlyCSE] Remove guards with conditions known to be true

Max Kazantsev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 26 23:31:48 PDT 2017


mkazantsev added inline comments.


================
Comment at: test/Transforms/EarlyCSE/guards.ll:6
 
-define i32 @test0(i32* %ptr, i1 %cond) {
+define i32 @test_00(i32* %ptr, i1 %cond) {
 ; We can do store to load forwarding over a guard, since it does not
----------------
reames wrote:
> minor: submit a separate patch without further review for the test name changes to simplify the diff.
Actually I've looked into other tests for EarlyCSE and decided to keep the naming as is for consistency.


https://reviews.llvm.org/D32476





More information about the llvm-commits mailing list