[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