Skip to content

Commit ee16a4b

Browse files
committed
address Tex's PR comments
1 parent 9cdfbc6 commit ee16a4b

File tree

1 file changed

+8
-9
lines changed

1 file changed

+8
-9
lines changed

llvm/lib/Transforms/Scalar/Scalarizer.cpp

Lines changed: 8 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -732,16 +732,15 @@ bool ScalarizerVisitor::splitCall(CallInst &CI) {
732732

733733
if (AreAllVectorsOfMatchingSize) {
734734
for (unsigned I = 1; I < CallType->getNumContainedTypes(); I++) {
735-
if (isVectorIntrinsicWithStructReturnOverloadAtField(ID, I)) {
736-
std::optional<VectorSplit> CurrVS = getVectorSplit(
737-
cast<FixedVectorType>(CallType->getContainedType(I)));
738-
// This case does not seem to happen, but it is possible for
739-
// VectorSplit.NumPacked >= NumElems. If that happens a VectorSplit
740-
// is not returned and we will bailout of handling this call.
741-
if (!CurrVS)
742-
return false;
735+
std::optional<VectorSplit> CurrVS =
736+
getVectorSplit(cast<FixedVectorType>(CallType->getContainedType(I)));
737+
// This case does not seem to happen, but it is possible for
738+
// VectorSplit.NumPacked >= NumElems. If that happens a VectorSplit
739+
// is not returned and we will bailout of handling this call.
740+
if (!CurrVS)
741+
return false;
742+
if (isVectorIntrinsicWithStructReturnOverloadAtField(ID, I))
743743
Tys.push_back(CurrVS->SplitTy);
744-
}
745744
}
746745
}
747746
// Assumes that any vector type has the same number of elements as the return

0 commit comments

Comments
 (0)