From ce2da1e6e4b41f1480c978f7571fdc0abc8ba32b Mon Sep 17 00:00:00 2001 From: Sanjay Patel Date: Tue, 12 Sep 2017 23:28:11 +0000 Subject: [PATCH] [SimplifyCFG] update test comments; NFC llvm-svn: 313090 --- llvm/test/Transforms/SimplifyCFG/div-rem-pairs.ll | 9 ++------- 1 file changed, 2 insertions(+), 7 deletions(-) diff --git a/llvm/test/Transforms/SimplifyCFG/div-rem-pairs.ll b/llvm/test/Transforms/SimplifyCFG/div-rem-pairs.ll index 85ffe1f4e0f3..4543d726930e 100644 --- a/llvm/test/Transforms/SimplifyCFG/div-rem-pairs.ll +++ b/llvm/test/Transforms/SimplifyCFG/div-rem-pairs.ll @@ -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: