[llvm] [LV] Enable considering higher VFs when data extend ops are present i… (PR #137593)

Sushant Gokhale via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 28 01:29:54 PDT 2025


sushgokh wrote:

> It sounds like you are looking for shouldMaximizeVectorBandwidth?

Maybe. Looking at the code for AArch64, I think it just considers if vector is FixedWidth. But other than that, there are no other conditions.
This patch adds the condition for this decision. Maybe this can be refined now or later.

> @huntergr-arm was working on enabling it for SVE but it can lead to issues with cost-modelling and perforance, which I believe were being worked through.

I tested the patch on Neoverse-v2 with SPEC17 and there are no regressions. Are there any specific cost-modelling/performance  issue you know of?

https://github.com/llvm/llvm-project/pull/137593


More information about the llvm-commits mailing list