[llvm] [GVN] Support rnflow pattern matching and transform (PR #162259)
Madhur Amilkanthwar via llvm-commits
llvm-commits at lists.llvm.org
Sun May 3 22:58:24 PDT 2026
madhur13490 wrote:
> AFAIK, there are still a couple of suggestions:
>
> * no being dependent on the specific condition - the transform cares that we choose one or another index, but not about the specific criteria involved in choosing it, right?
Thanks @momchil for reminding me. I have updated the patch to
* made it more generic and that should handle your's and @nikic comments. Summarized the predicate generalisation (filter removed; LHS-matched case canonicalized via swapOperands()
* covered by the two new positive tests
For second suggestion,
> * what if the indices are `i32` or `i64` throughout with no conversion shenanigans?
I would like make that further generalization as a separate patch as the transform is already a big piece. Keeping this patch simple and make the next one incremental will make the work manageable.
Ping to @nikic and @alinas to also take a look.
https://github.com/llvm/llvm-project/pull/162259
More information about the llvm-commits
mailing list