[PATCH] D36784: [InstCombine] Teach canEvaluateTruncated to handle arithmetic shift (including those with vector splat shift amount)

Amjad Aboud via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 16 14:40:32 PDT 2017


aaboud added a comment.

In https://reviews.llvm.org/D36784#843566, @spatel wrote:

> LGTM.
>
> Just to make sure I'm understanding: the multiply patch (https://reviews.llvm.org/D36679) is independent of this?


Yes, I managed to find an example where I am using OR instruction and that AShr instruction cannot be replaced by LShr instruction.
So, we can commit https://reviews.llvm.org/D36679 separately after this one.


https://reviews.llvm.org/D36784





More information about the llvm-commits mailing list