[llvm] [LV] Don't skip VPlan cost model for div/rem instructions (PR #187056)
Florian Hahn via llvm-commits
llvm-commits at lists.llvm.org
Mon Mar 23 14:08:28 PDT 2026
fhahn wrote:
> > I did run this on a set of workloads and it looks like there's some divergences at least on AAarch64. Will prepare a reproducer soon.
>
> Any update here @fhahn? I've improved the cost model to better reflect the VPlan model and the generated LLVM IR. I have more patches to follow in this area that are currently gated on this patch.
Sorry took a bit longer to get a good reproducer. It looks like a separate issue to insert/extract costs. I think with https://github.com/llvm/llvm-project/pull/188126, the version of the PR without the latest change didn't trigger any crashes with a set of different configs in my testing. I've not yet tried with the latest change, but it may be worth splitting it up?
https://github.com/llvm/llvm-project/pull/187056
More information about the llvm-commits
mailing list