[llvm] r324831 - [X86] Remove some redundant qualifications from the setOperationAction blocks. NFC
Craig Topper via llvm-commits
llvm-commits at lists.llvm.org
Sat Feb 10 19:07:19 PST 2018
Author: ctopper
Date: Sat Feb 10 19:07:19 2018
New Revision: 324831
URL: http://llvm.org/viewvc/llvm-project?rev=324831&view=rev
Log:
[X86] Remove some redundant qualifications from the setOperationAction blocks. NFC
These were added as part of the refactoring for prefer vector width. At the time I thought the hasAVX512 here would be replaced with "allow 512 bit vectors" so that it would read "allow 512 bit vectors OR VLX". But now the plan is to only give the option of disabling 512 bit vectors when VLX is enabled. So we don't need this qualification at all
Modified:
llvm/trunk/lib/Target/X86/X86ISelLowering.cpp
Modified: llvm/trunk/lib/Target/X86/X86ISelLowering.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/X86/X86ISelLowering.cpp?rev=324831&r1=324830&r2=324831&view=diff
==============================================================================
--- llvm/trunk/lib/Target/X86/X86ISelLowering.cpp (original)
+++ llvm/trunk/lib/Target/X86/X86ISelLowering.cpp Sat Feb 10 19:07:19 2018
@@ -1361,8 +1361,7 @@ X86TargetLowering::X86TargetLowering(con
}
}// has AVX-512
- if (!Subtarget.useSoftFloat() &&
- (Subtarget.hasAVX512() || Subtarget.hasVLX())) {
+ if (!Subtarget.useSoftFloat() && Subtarget.hasAVX512()) {
// These operations are handled on non-VLX by artificially widening in
// isel patterns.
// TODO: Custom widen in lowering on non-VLX and drop the isel patterns?
@@ -1514,8 +1513,7 @@ X86TargetLowering::X86TargetLowering(con
}
}
- if (!Subtarget.useSoftFloat() && Subtarget.hasBWI() &&
- (Subtarget.hasAVX512() || Subtarget.hasVLX())) {
+ if (!Subtarget.useSoftFloat() && Subtarget.hasBWI()) {
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);
More information about the llvm-commits
mailing list