[llvm] [LoopUnroll] Introduce parallel reduction phis when unrolling. (PR #149470)

Florian Hahn via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 22 05:04:03 PDT 2025


fhahn wrote:

> I was imprecise when I said you "can't unroll" that loop; technically you can unroll almost anything. But your current version of reduction detection won't work for that loop, and making reduction detection work would require significant infrastructure changes to unrolling, to introduce runtime checks.

Ah sorry, the initial version of the patch was missing a check for reductions with intermediate stores, so it incorrectly introduced parallel reduction phis for the loop. Should be fixed in the latest version + test added, thanks

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


More information about the llvm-commits mailing list