[PATCH] D24154: [DAGCombiner] Fix incorrect sinking of a truncate into the operand of a shift.

Andrea Di Biagio via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 1 16:49:28 PDT 2016


andreadb added a comment.

In https://reviews.llvm.org/D24154#532078, @arsenm wrote:

> LGTM, but can you add a scalar case with the original test I added for this which hits the new limit of the full scalar bit width behavior instead of / 2


Thanks for the quick review.
Sure, I will add extra scalar test to reduce-trunc-shl.ll.
I can add tests that are similar to those that you have added in test AMDGPU/shift-i64-opts.ll. I will also add tests for the case where the shift count is equal to the scala size in bits.

Cheers,
Andrea


https://reviews.llvm.org/D24154





More information about the llvm-commits mailing list