]> granicus.if.org Git - llvm/commit
[Thumb] Teach ISel how to lower compares of AND bitmasks efficiently
authorJames Molloy <james.molloy@arm.com>
Thu, 3 Nov 2016 10:18:20 +0000 (10:18 +0000)
committerJames Molloy <james.molloy@arm.com>
Thu, 3 Nov 2016 10:18:20 +0000 (10:18 +0000)
commite03e2fa99dff50706281736731ec0ccc4393b1a3
treec18f2cb491ec70cec138b907bde8783f9b1b2629
parentd6479398a02b3816e58615ce932abd71cf20f9cf
[Thumb] Teach ISel how to lower compares of AND bitmasks efficiently

This recommits r281323, which was backed out for two reasons. One, a selfhost failure, and two, it apparently caused Chromium failures. Actually, the latter was a red herring. The log has expired from the former, but I suspect that was a red herring too (actually caused by another problematic patch of mine). Therefore reapplying, and will watch the bots like a hawk.

For the common pattern (CMPZ (AND x, #bitmask), #0), we can do some more efficient instruction selection if the bitmask is one consecutive sequence of set bits (32 - clz(bm) - ctz(bm) == popcount(bm)).

1) If the bitmask touches the LSB, then we can remove all the upper bits and set the flags by doing one LSLS.
2) If the bitmask touches the MSB, then we can remove all the lower bits and set the flags with one LSRS.
3) If the bitmask has popcount == 1 (only one set bit), we can shift that bit into the sign bit with one LSLS and change the condition query from NE/EQ to MI/PL (we could also implement this by shifting into the carry bit and branching on BCC/BCS).
4) Otherwise, we can emit a sequence of LSLS+LSRS to remove the upper and lower zero bits of the mask.

1-3 require only one 16-bit instruction and can elide the CMP. 4 requires two 16-bit instructions but can elide the CMP and doesn't require materializing a complex immediate, so is also a win.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@285893 91177308-0d34-0410-b5e6-96231b3b80d8
lib/Target/ARM/ARMBaseInstrInfo.cpp
lib/Target/ARM/ARMISelDAGToDAG.cpp
test/CodeGen/ARM/and-cmpz.ll [new file with mode: 0644]
test/CodeGen/ARM/arm-and-tst-peephole.ll
test/CodeGen/ARM/arm-shrink-wrapping.ll
test/CodeGen/ARM/call-tc.ll
test/CodeGen/ARM/debug-info-branch-folding.ll
test/CodeGen/Thumb/thumb-shrink-wrapping.ll
test/CodeGen/Thumb2/float-ops.ll