diff options
| author | Craig Topper <craig.topper@intel.com> | 2017-12-18 04:50:05 +0000 |
|---|---|---|
| committer | Craig Topper <craig.topper@intel.com> | 2017-12-18 04:50:05 +0000 |
| commit | 8e2837cc6e0bc07d0f2722aa277a2c433946df05 (patch) | |
| tree | e05793ea013f049e6d5e9a84c2cbb4667042996e /llvm/lib | |
| parent | b0b67a8d38643f74c4d748cdbfdc8d78cd857319 (diff) | |
| download | bcm5719-llvm-8e2837cc6e0bc07d0f2722aa277a2c433946df05.tar.gz bcm5719-llvm-8e2837cc6e0bc07d0f2722aa277a2c433946df05.zip | |
[X86] Fix mistake that I made when splitting up the setOperationAction calls recently.
The block I moved things that need BWI and 512-bit or VLX is incorrectly qualified with just hasBWI || hasVLX. Here I've qualified it with hasBWI && (hasAVX512 || hasVLX) where the hasAVX512 will be replaced with allowing 512-bit vectors in an upcoming patch.
llvm-svn: 320957
Diffstat (limited to 'llvm/lib')
| -rw-r--r-- | llvm/lib/Target/X86/X86ISelLowering.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/llvm/lib/Target/X86/X86ISelLowering.cpp b/llvm/lib/Target/X86/X86ISelLowering.cpp index 4049d9a0926..a72f4daa5e1 100644 --- a/llvm/lib/Target/X86/X86ISelLowering.cpp +++ b/llvm/lib/Target/X86/X86ISelLowering.cpp @@ -1502,8 +1502,8 @@ X86TargetLowering::X86TargetLowering(const X86TargetMachine &TM, } } - if (!Subtarget.useSoftFloat() && - (Subtarget.hasBWI() || Subtarget.hasVLX())) { + if (!Subtarget.useSoftFloat() && Subtarget.hasBWI() && + (Subtarget.hasAVX512() || Subtarget.hasVLX())) { for (auto VT : { MVT::v32i8, MVT::v16i8, MVT::v16i16, MVT::v8i16 }) { setOperationAction(ISD::MLOAD, VT, Subtarget.hasVLX() ? Legal : Custom); setOperationAction(ISD::MSTORE, VT, Subtarget.hasVLX() ? Legal : Custom); |
