[llvm] [LV, VP]VP intrinsics support for the Loop Vectorizer + adding new tail-folding mode using EVL. (PR #76172)

Florian Hahn via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 28 13:54:29 PDT 2024


fhahn wrote:

> > It looks like there's a UBSan failure when running some of the tests (comment inline, looks unrelated to the patch, but the tests need to force a vector width > 1), would be good to check with an UBSan build.
> 
> Possible, will double check.
> 
> > Also, the target-independent tests don't exercise the EVL code path due to missing support for VP intrinsics at the moment AFACIT; I think we need to have them with `RISCV` targets as well, as otherwise some code paths aren't exercised in the EVL code path (e.g. interleave groups, gather/scatter, reverse memory ops, masked ops)
> 
> I had all these tests under RISCV. You asked to make them target independent. Should I again move them to RISCV?

I'd just move the individual files to RISCV. Earlier comment was if this would retain test coverage (i.e. by adding a something similar like `-force-target-supports-scalable-vectors` for vector-predication), sorry that wasn't clearer from the previous comment

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


More information about the llvm-commits mailing list