]> granicus.if.org Git - llvm/commitdiff
[SimplifyCFG] update test comments; NFC
authorSanjay Patel <spatel@rotateright.com>
Tue, 12 Sep 2017 23:28:11 +0000 (23:28 +0000)
committerSanjay Patel <spatel@rotateright.com>
Tue, 12 Sep 2017 23:28:11 +0000 (23:28 +0000)
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@313090 91177308-0d34-0410-b5e6-96231b3b80d8

test/Transforms/SimplifyCFG/div-rem-pairs.ll

index 85ffe1f4e0f37e61146b35370ac1bfd85cec64e7..4543d726930ee4cfcfe689f86541665a338748ce 100644 (file)
@@ -1,7 +1,8 @@
 ; RUN: opt -simplifycfg -S < %s | FileCheck %s
 
-; FIXME: Hoist the sdiv because it's safe and free.
+; We could hoist the div/rem in these tests because it's safe to do so.
 ; PR31028 - https://bugs.llvm.org/show_bug.cgi?id=31028
+; ...but since there's a separate pass for that, don't bother.
 
 define i32 @hoist_sdiv(i32 %a, i32 %b) {
 ; CHECK-LABEL: @hoist_sdiv(
@@ -30,8 +31,6 @@ end:
   ret i32 %ret
 }
 
-; FIXME: Hoist the udiv because it's safe and free.
-
 define i64 @hoist_udiv(i64 %a, i64 %b) {
 ; CHECK-LABEL: @hoist_udiv(
 ; CHECK-NEXT:  entry:
@@ -59,8 +58,6 @@ end:
   ret i64 %ret
 }
 
-; FIXME: Hoist the srem because it's safe and likely free.
-
 define i16 @hoist_srem(i16 %a, i16 %b) {
 ; CHECK-LABEL: @hoist_srem(
 ; CHECK-NEXT:  entry:
@@ -88,8 +85,6 @@ end:
   ret i16 %ret
 }
 
-; FIXME: Hoist the urem because it's safe and likely free.
-
 define i8 @hoist_urem(i8 %a, i8 %b) {
 ; CHECK-LABEL: @hoist_urem(
 ; CHECK-NEXT:  entry: