diff options
| author | Craig Topper <craig.topper@gmail.com> | 2017-03-12 22:29:12 +0000 |
|---|---|---|
| committer | Craig Topper <craig.topper@gmail.com> | 2017-03-12 22:29:12 +0000 |
| commit | 7d56c8315b09ee87dae02735f0ed87d56d035dea (patch) | |
| tree | 9eee74865df254d3f80c9efc23713f058ad0a7dd /llvm/lib/Target/X86/X86ISelLowering.cpp | |
| parent | 9625db09c1ae7163b96b4c306ae28f2f2abdfa7f (diff) | |
| download | bcm5719-llvm-7d56c8315b09ee87dae02735f0ed87d56d035dea.tar.gz bcm5719-llvm-7d56c8315b09ee87dae02735f0ed87d56d035dea.zip | |
[AVX-512] Fix the valid immediates for the scatter/gather prefetch intrinsics.
The immediate should be 1 or 2, not 0 or 1. This was found while adding bounds checking to clang. In fact the existing clang builtin test failed if we ran it all the way to assembly.
llvm-svn: 297591
Diffstat (limited to 'llvm/lib/Target/X86/X86ISelLowering.cpp')
| -rw-r--r-- | llvm/lib/Target/X86/X86ISelLowering.cpp | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/llvm/lib/Target/X86/X86ISelLowering.cpp b/llvm/lib/Target/X86/X86ISelLowering.cpp index b74bf9a947c..cf8304e565a 100644 --- a/llvm/lib/Target/X86/X86ISelLowering.cpp +++ b/llvm/lib/Target/X86/X86ISelLowering.cpp @@ -20203,8 +20203,9 @@ static SDValue LowerINTRINSIC_W_CHAIN(SDValue Op, const X86Subtarget &Subtarget, case PREFETCH: { SDValue Hint = Op.getOperand(6); unsigned HintVal = cast<ConstantSDNode>(Hint)->getZExtValue(); - assert(HintVal < 2 && "Wrong prefetch hint in intrinsic: should be 0 or 1"); - unsigned Opcode = (HintVal ? IntrData->Opc1 : IntrData->Opc0); + assert((HintVal == 1 || HintVal == 2) && + "Wrong prefetch hint in intrinsic: should be 1 or 2"); + unsigned Opcode = (HintVal != 1 ? IntrData->Opc1 : IntrData->Opc0); SDValue Chain = Op.getOperand(0); SDValue Mask = Op.getOperand(2); SDValue Index = Op.getOperand(3); |
