[llvm] [LV] Autovectorization for the all-in-one histogram intrinsic (PR #91458)

Florian Hahn via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 11 08:03:05 PDT 2024


https://github.com/fhahn commented:

Adding some more comments inline, I think the current version is introducing crashes in a number of cases (when unrolling and possibly epilogue vectorization).

It would also be good to add additional test coverage, including missing negative tests for the memory analysis, SVE-independent tests and a test to check VPlan printing.

As there are a number of potential issues, it might be good to revert while addressing those

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


More information about the llvm-commits mailing list