Skip to content

Commit fe780fe

Browse files
committed
Address review comment
1 parent b25a186 commit fe780fe

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

llvm/lib/Target/AArch64/AArch64TargetTransformInfo.cpp

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -4557,24 +4557,24 @@ static bool isLoopSizeWithinBudget(Loop *L, AArch64TTIImpl &TTI,
45574557
InstructionCost LoopCost = 0;
45584558

45594559
if (findStringMetadataForLoop(L, "llvm.loop.isvectorized"))
4560-
return 0;
4560+
return false;
45614561

45624562
for (auto *BB : L->getBlocks()) {
45634563
for (auto &I : *BB) {
45644564
if (!shouldUnrollLoopWithInstruction(I, TTI))
4565-
return 0;
4565+
return false;
45664566

45674567
SmallVector<const Value *, 4> Operands(I.operand_values());
45684568
InstructionCost Cost =
45694569
TTI.getInstructionCost(&I, Operands, TTI::TCK_CodeSize);
45704570
// This can happen with intrinsics that don't currently have a cost model
45714571
// or for some operations that require SVE.
45724572
if (!Cost.isValid())
4573-
return 0;
4573+
return false;
45744574

45754575
LoopCost += Cost;
45764576
if (LoopCost > Budget)
4577-
return 0;
4577+
return false;
45784578
}
45794579
}
45804580

0 commit comments

Comments
 (0)