[llvm] [LV] Change loops' interleave count computation (PR #73766)
Nilanjana Basu via llvm-commits
llvm-commits at lists.llvm.org
Wed Dec 6 18:43:55 PST 2023
================
@@ -5737,10 +5743,19 @@ LoopVectorizationCostModel::selectInterleaveCount(ElementCount VF,
// the InterleaveCount as if vscale is '1', although if some information about
// the vector is known (e.g. min vector size), we can make a better decision.
if (BestKnownTC) {
- MaxInterleaveCount =
- std::min(*BestKnownTC / VF.getKnownMinValue(), MaxInterleaveCount);
- // Make sure MaxInterleaveCount is greater than 0.
- MaxInterleaveCount = std::max(1u, MaxInterleaveCount);
+ unsigned EstimatedVF = VF.getKnownMinValue();
+ if (VF.isScalable()) {
+ if (std::optional<unsigned> VScale = getVScaleForTuning(L, TTI))
+ EstimatedVF *= *VScale;
+ }
+ if (InterleaveSmallLoopScalarReduction || (*BestKnownTC % EstimatedVF == 0))
----------------
nilanjana87 wrote:
> Why the special handling for InterleaveSmallLoopScalarReduction?
Removed the special handling for `InterleaveSmallLoopScalarReduction`. Had kept it earlier to not change its behavior as it seemed like a special-purpose flag.
https://github.com/llvm/llvm-project/pull/73766
More information about the llvm-commits
mailing list