[llvm] r373245 - [X86] Address post-commit review from code I accidentally commited in r373136.
Craig Topper via llvm-commits
llvm-commits at lists.llvm.org
Mon Sep 30 11:43:27 PDT 2019
Author: ctopper
Date: Mon Sep 30 11:43:27 2019
New Revision: 373245
URL: http://llvm.org/viewvc/llvm-project?rev=373245&view=rev
Log:
[X86] Address post-commit review from code I accidentally commited in r373136.
See https://reviews.llvm.org/D68167
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=373245&r1=373244&r2=373245&view=diff
==============================================================================
--- llvm/trunk/lib/Target/X86/X86ISelLowering.cpp (original)
+++ llvm/trunk/lib/Target/X86/X86ISelLowering.cpp Mon Sep 30 11:43:27 2019
@@ -43479,16 +43479,19 @@ static SDValue combineAdd(SDNode *N, Sel
// FIXME: We have the (sub Y, (zext (vXi1 X))) -> (add (sext (vXi1 X)), Y) in
// generic DAG combine without a legal type check, but adding this there
// caused regressions.
- if (Subtarget.hasAVX512() && VT.isVector()) {
+ if (VT.isVector()) {
+ const TargetLowering &TLI = DAG.getTargetLoweringInfo();
if (Op0.getOpcode() == ISD::ZERO_EXTEND &&
- Op0.getOperand(0).getValueType().getVectorElementType() == MVT::i1) {
+ Op0.getOperand(0).getValueType().getVectorElementType() == MVT::i1 &&
+ TLI.isTypeLegal(Op0.getOperand(0).getValueType())) {
SDLoc DL(N);
SDValue SExt = DAG.getNode(ISD::SIGN_EXTEND, DL, VT, Op0.getOperand(0));
return DAG.getNode(ISD::SUB, DL, VT, Op1, SExt);
}
if (Op1.getOpcode() == ISD::ZERO_EXTEND &&
- Op1.getOperand(0).getValueType().getVectorElementType() == MVT::i1) {
+ Op1.getOperand(0).getValueType().getVectorElementType() == MVT::i1 &&
+ TLI.isTypeLegal(Op1.getOperand(0).getValueType())) {
SDLoc DL(N);
SDValue SExt = DAG.getNode(ISD::SIGN_EXTEND, DL, VT, Op1.getOperand(0));
return DAG.getNode(ISD::SUB, DL, VT, Op0, SExt);
More information about the llvm-commits
mailing list