Skip to content

Commit fda85a1

Browse files
authored
[DebugInfo][LoopVectorizer][NFC] Use unknown annotations for more instructions (#170522)
Some recent patches have added more non-annotated empty locations to the loop vectorizer, resulting in errors reported on the DebugLoc coverage tracking buildbot: https://lab.llvm.org/staging/#/builders/222/builds/1938 This patch adds "unknown" annotations in place of the empty locations, allowing the buildbot to ignore them for now.
1 parent e84fdbe commit fda85a1

File tree

1 file changed

+7
-6
lines changed

1 file changed

+7
-6
lines changed

llvm/lib/Transforms/Vectorize/VPlanTransforms.cpp

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -942,7 +942,8 @@ static VPValue *optimizeLatchExitInductionUser(
942942
VPValue *Step = WideIV->getStepValue();
943943
Type *ScalarTy = TypeInfo.inferScalarType(WideIV);
944944
if (ScalarTy->isIntegerTy())
945-
return B.createNaryOp(Instruction::Sub, {EndValue, Step}, {}, "ind.escape");
945+
return B.createNaryOp(Instruction::Sub, {EndValue, Step},
946+
DebugLoc::getUnknown(), "ind.escape");
946947
if (ScalarTy->isPointerTy()) {
947948
Type *StepTy = TypeInfo.inferScalarType(Step);
948949
auto *Zero = Plan.getConstantInt(StepTy, 0);
@@ -3748,11 +3749,11 @@ void VPlanTransforms::handleUncountableEarlyExit(VPBasicBlock *EarlyExitingVPBB,
37483749
if (!IncomingFromEarlyExit->isLiveIn()) {
37493750
// Update the incoming value from the early exit.
37503751
VPValue *FirstActiveLane = EarlyExitB.createNaryOp(
3751-
VPInstruction::FirstActiveLane, {CondToEarlyExit}, nullptr,
3752-
"first.active.lane");
3752+
VPInstruction::FirstActiveLane, {CondToEarlyExit},
3753+
DebugLoc::getUnknown(), "first.active.lane");
37533754
IncomingFromEarlyExit = EarlyExitB.createNaryOp(
37543755
VPInstruction::ExtractLane, {FirstActiveLane, IncomingFromEarlyExit},
3755-
nullptr, "early.exit.value");
3756+
DebugLoc::getUnknown(), "early.exit.value");
37563757
ExitIRI->setOperand(EarlyExitIdx, IncomingFromEarlyExit);
37573758
}
37583759
}
@@ -4968,8 +4969,8 @@ void VPlanTransforms::updateScalarResumePhis(
49684969
"Cannot handle loops with uncountable early exits");
49694970
if (IsFOR)
49704971
ResumeFromVectorLoop = MiddleBuilder.createNaryOp(
4971-
VPInstruction::ExtractLastElement, {ResumeFromVectorLoop}, {},
4972-
"vector.recur.extract");
4972+
VPInstruction::ExtractLastElement, {ResumeFromVectorLoop},
4973+
DebugLoc::getUnknown(), "vector.recur.extract");
49734974
ResumePhiR->setName(IsFOR ? "scalar.recur.init" : "bc.merge.rdx");
49744975
ResumePhiR->setOperand(0, ResumeFromVectorLoop);
49754976
}

0 commit comments

Comments
 (0)