[llvm-branch-commits] [llvm] [RISCV] Attach VLMAX range attribute for vsetvli/vsetvlimax in InstCombine (PR #218652)

[Sam Elliott via llvm-branch-commits]

lenary wrote:

Can we check what this callback is doing against the guidance here and the other targets: https://llvm.org/docs/InstCombineContributorGuide.html#canonicalization-and-target-independence

It's not clear to me how much we can take subtarget features into account when doing these combines. AArch64 seems to take into account the presence or absence of streaming mode in their checks (`instCombineSMECntsd`), so maybe this is equally ok for us to take into account the vlmin from target features.

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