[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