From: Richard Sandiford Date: Mon, 13 Jan 2014 15:40:25 +0000 (+0000) Subject: [SystemZ] Improve risbg-01.ll test X-Git-Url: http://demsky.eecs.uci.edu/git/?a=commitdiff_plain;h=be110d99db9323ef2f484deca230c14d8ee0e267;p=oota-llvm.git [SystemZ] Improve risbg-01.ll test The old mask in f24 wasn't well chosen because the lshr would always be zero. CodeGen didn't detect this but InstCombine would. The new mask ensures that both shifts are needed. f26 is specifically testing for a wrap-around mask. The AND can be applied to just the shift left, either before or after the shift. Again, CodeGen kept it in the original form but InstCombine would mask after the shift instead. The exact choice of NILF isn't important for the test so I just dropped it and kept the rotate. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@199115 91177308-0d34-0410-b5e6-96231b3b80d8 --- diff --git a/test/CodeGen/SystemZ/risbg-01.ll b/test/CodeGen/SystemZ/risbg-01.ll index e303067e539..d75e8e4b11a 100644 --- a/test/CodeGen/SystemZ/risbg-01.ll +++ b/test/CodeGen/SystemZ/risbg-01.ll @@ -269,12 +269,12 @@ define i64 @f23(i64 %foo) { ; mask and rotate. define i32 @f24(i32 %foo) { ; CHECK-LABEL: f24: -; CHECK: nilf %r2, 14 -; CHECK: rll %r2, %r2, 3 +; CHECK: nilf %r2, 254 +; CHECK: rll %r2, %r2, 29 ; CHECK: br %r14 - %and = and i32 %foo, 14 - %parta = shl i32 %and, 3 - %partb = lshr i32 %and, 29 + %and = and i32 %foo, 254 + %parta = lshr i32 %and, 3 + %partb = shl i32 %and, 29 %rotl = or i32 %parta, %partb ret i32 %rotl } @@ -295,7 +295,6 @@ define i64 @f25(i64 %foo) { ; This again needs a separate mask and rotate. define i32 @f26(i32 %foo) { ; CHECK-LABEL: f26: -; CHECK: nill %r2, 65487 ; CHECK: rll %r2, %r2, 5 ; CHECK: br %r14 %and = and i32 %foo, -49