[llvm] [LoopVectorize] Further improve cost model for early exit loops (PR #126235)
Florian Hahn via llvm-commits
llvm-commits at lists.llvm.org
Mon Mar 10 03:31:22 PDT 2025
https://github.com/fhahn commented:
> Hi @fhahn, could you clarify what you mean by passing in `CostCtx`? I have addressed most of your review comments downstream, but it's not clear whether you want me to just pass it into `calculateEarlyExitCost` and create it in `isOutsideLoopWorkProfitable`, or pass it to `isOutsideLoopWorkProfitable`. Once I know your preference I can upload a new version - thanks!
My thinking was pass to `isOutsideLoopWorkProfitable` instead of the legacy cost model, to avoid spreading new references of it
https://github.com/llvm/llvm-project/pull/126235
More information about the llvm-commits
mailing list