]> granicus.if.org Git - llvm/commit
[InstCombine] Dropping redundant masking before left-shift [5/5] (PR42563)
authorRoman Lebedev <lebedev.ri@gmail.com>
Fri, 19 Jul 2019 08:26:58 +0000 (08:26 +0000)
committerRoman Lebedev <lebedev.ri@gmail.com>
Fri, 19 Jul 2019 08:26:58 +0000 (08:26 +0000)
commit634dca6e2d7efa19277d7dd17f32b4d78afde8f1
treeb186d9dd8b9720b1e8ac934fb70fed0790aa177d
parent3e066579ec3ab41bee2bbb79079a60fe962b90f6
[InstCombine] Dropping redundant masking before left-shift [5/5] (PR42563)

Summary:
If we have some pattern that leaves only some low bits set, and then performs
left-shift of those bits, if none of the bits that are left after the final
shift are modified by the mask, we can omit the mask.

There are many variants to this pattern:
f. `((x << MaskShAmt) a>> MaskShAmt) << ShiftShAmt`
All these patterns can be simplified to just:
`x << ShiftShAmt`
iff:
f. `(ShiftShAmt-MaskShAmt) s>= 0` (i.e. `ShiftShAmt u>= MaskShAmt`)

Normally, the inner pattern is sign-extend,
but for our purposes it's no different to other patterns:

alive proofs:
f: https://rise4fun.com/Alive/7U3

For now let's start with patterns where both shift amounts are variable,
with trivial constant "offset" between them, since i believe this is
both simplest to handle and i think this is most common.
But again, there are likely other variants where we could use
ValueTracking/ConstantRange to handle more cases.

https://bugs.llvm.org/show_bug.cgi?id=42563

Differential Revision: https://reviews.llvm.org/D64524

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@366540 91177308-0d34-0410-b5e6-96231b3b80d8
lib/Transforms/InstCombine/InstCombineShifts.cpp
test/Transforms/InstCombine/redundant-left-shift-input-masking-variant-f.ll