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

[Igor Breger via llvm-commits]

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.


Repository:
  rL LLVM

http://reviews.llvm.org/D20897