[llvm] r326170 - [X86] Simplify if condition. NFC
Craig Topper via llvm-commits
llvm-commits at lists.llvm.org
Mon Feb 26 22:00:38 PST 2018
Author: ctopper
Date: Mon Feb 26 22:00:38 2018
New Revision: 326170
URL: http://llvm.org/viewvc/llvm-project?rev=326170&view=rev
Log:
[X86] Simplify if condition. NFC
SSE2 implies SSE1 and we already covered f32 in the SSE1 check so we don't need to check f32 in the SSE2 check.
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=326170&r1=326169&r2=326170&view=diff
==============================================================================
--- llvm/trunk/lib/Target/X86/X86ISelLowering.cpp (original)
+++ llvm/trunk/lib/Target/X86/X86ISelLowering.cpp Mon Feb 26 22:00:38 2018
@@ -18434,7 +18434,7 @@ SDValue X86TargetLowering::LowerSELECT(S
// are available or VBLENDV if AVX is available.
// Otherwise FP cmovs get lowered into a less efficient branch sequence later.
if (Cond.getOpcode() == ISD::SETCC &&
- ((Subtarget.hasSSE2() && (VT == MVT::f32 || VT == MVT::f64)) ||
+ ((Subtarget.hasSSE2() && VT == MVT::f64) ||
(Subtarget.hasSSE1() && VT == MVT::f32)) &&
VT == Cond.getOperand(0).getSimpleValueType() && Cond->hasOneUse()) {
SDValue CondOp0 = Cond.getOperand(0), CondOp1 = Cond.getOperand(1);
More information about the llvm-commits
mailing list