[PATCH] D28171: Enable disabled loopidiom test. Apparently we handle it now

[Xin Tong via Phabricator via llvm-commits]

This revision was automatically updated to reflect the committed changes.
Closed by commit rL290900: Enable disabled loopidiom test. Apparently we handle it now (authored by trentxintong).

Changed prior to commit:
  https://reviews.llvm.org/D28171?vs=82712&id=82918#toc

Repository:
  rL LLVM

https://reviews.llvm.org/D28171

Files:
  llvm/trunk/test/Transforms/LoopIdiom/basic.ll


Index: llvm/trunk/test/Transforms/LoopIdiom/basic.ll
===================================================================
--- llvm/trunk/test/Transforms/LoopIdiom/basic.ll
+++ llvm/trunk/test/Transforms/LoopIdiom/basic.ll
@@ -97,8 +97,7 @@
 ; CHECK: ret void
 }
 
-
-;; TODO: We should be able to promote this memset.  Not yet though.
+; Make sure the first store in the loop is turned into a memset.
 define void @test4(i8* %Base) nounwind ssp {
 bb.nph:                                           ; preds = %entry
   %Base100 = getelementptr i8, i8* %Base, i64 1000
@@ -118,9 +117,8 @@
 
 for.end:                                          ; preds = %for.body, %entry
   ret void
-; CHECK-TODO-LABEL: @test4(
-; CHECK-TODO: call void @llvm.memset.p0i8.i64(i8* %Base, i8 0, i64 100, i32 1, i1 false)
-; CHECK-TODO-NOT: store
+; CHECK-LABEL: @test4(
+; CHECK: call void @llvm.memset.p0i8.i64(i8* %Base, i8 0, i64 100, i32 1, i1 false)
 }
 
 ; This can't be promoted: the memset is a store of a loop variant value.


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D28171.82918.patch
Type: text/x-patch
Size: 1011 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170103/70e8c80a/attachment.bin>