Skip to content

Commit

Permalink
[AVX512] No need to check for BWI being enabled before lowering v32i1…
Browse files Browse the repository at this point in the history
…6 and v64i8 shuffles. If we get this far the types are already legal which means BWI must be enabled.

llvm-svn: 272449
  • Loading branch information
topperc committed Jun 11, 2016
1 parent 334aa07 commit b9b86fc
Showing 1 changed file with 2 additions and 9 deletions.
11 changes: 2 additions & 9 deletions llvm/lib/Target/X86/X86ISelLowering.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -11889,20 +11889,13 @@ static SDValue lower512BitVectorShuffle(SDValue Op, SDValue V1, SDValue V2,
case MVT::v16i32:
return lowerV16I32VectorShuffle(Op, V1, V2, Subtarget, DAG);
case MVT::v32i16:
if (Subtarget.hasBWI())
return lowerV32I16VectorShuffle(Op, V1, V2, Subtarget, DAG);
break;
return lowerV32I16VectorShuffle(Op, V1, V2, Subtarget, DAG);
case MVT::v64i8:
if (Subtarget.hasBWI())
return lowerV64I8VectorShuffle(Op, V1, V2, Subtarget, DAG);
break;
return lowerV64I8VectorShuffle(Op, V1, V2, Subtarget, DAG);

default:
llvm_unreachable("Not a valid 512-bit x86 vector type!");
}

// Otherwise fall back on splitting.
return splitAndLowerVectorShuffle(DL, VT, V1, V2, Mask, DAG);
}

// Lower vXi1 vector shuffles.
Expand Down

0 comments on commit b9b86fc

Please sign in to comment.