[PATCH] D20897: [AVX512/AVX][Intrinsics] Fix Variable Bit Shift Right Arithmetic intrinsic lowering.

Igor Breger via llvm-commits llvm-commits at lists.llvm.org
Sun Jun 19 06:44:26 PDT 2016

igorb added a comment.

In http://reviews.llvm.org/D20897#461603, @RKSimon wrote:

> Thanks Igor - do you think the logical shift equivalents need to be changed as well? They also have defined behaviour for out of range shifts.

I am not sure . The out of range shifts behaviour undefined but implemented as "undefined results" to be always  0.
 APInt APInt::lshr(unsigned shiftAmt)
 APInt LLVM_ATTRIBUTE_UNUSED_RESULT shl(unsigned shiftAmt)

I will add a few tests to insure that this behavior doesn't change.



More information about the llvm-commits mailing list