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

Madhur Amilkanthwar via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 17 22:21:41 PDT 2025


madhur13490 wrote:

> > Please note, this patch offers 4x speedup for the pattern.
> 
> I do believe this to be true, but equally I think you also have to accept the risk of a 4x slowdown once this is finally supported in GVN as the patterns will no longer match. If this PR does go ahead, then it might be a good idea first to add similar tests in GVN. That way you have a chance at getting an early warning that work in GVN will put this idiom at risk.

Yes, agree with you. I will add the test in GVN soon.

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


More information about the llvm-commits mailing list