[llvm] [AArch64][LoopIdiom] Generalize AArch64LoopIdiomTransform into LoopIdiomTransform (PR #94081)

Min-Yih Hsu via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 3 16:02:34 PDT 2024


mshockwave wrote:

> Can we please split this into two changes, one which does the move and rename, and a second which adds the new functionality? This diff is slightly confusing as a reader.

I don't think I add any new functionalities in this patch, the support for RVV-style vectorization is in #94082 

> 
> Naming wise, maybe LoopIdiomVectorizer? And update the comment to be really explicit about the goal being to eventually kill this off as the vectorizer grows more powerful?

I second for LoopIdiomVectorizer.

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


More information about the llvm-commits mailing list