[llvm] [LoopIdiomVectorize] Recognize and transform minidx pattern (PR #144987)

Sjoerd Meijer via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 10 05:34:14 PDT 2025


sjoerdmeijer wrote:

> We can't block good improvements in earlier scalar passes based on the fact we no longer recognise the idiom here.

Agreed, of course. And this pattern matching needs to disappear anyway, long term. 

I guess the main thing left to assess and investigate whether this short term solution can be salvaged is what @fhahn wrote about legality:

> it looks like there are a number of legality checks missing, e.g. it doesn't check if there are any potential memory writes in the loop, and possibly misses checks if PRE is valid

The other comments look minor. 

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


More information about the llvm-commits mailing list