[PATCH] D76363: [MLIR] Add parallel loop coalescing.

[Theodore Popp via Phabricator via llvm-commits]

tpopp marked an inline comment as done.
tpopp added inline comments.


================
Comment at: mlir/lib/Transforms/Utils/LoopUtils.cpp:1163
+
+      Value iv = (idx == e - 1)
+                     ? previous
----------------
herhut wrote:
> It would read easier for me if updating previous was also done here except for the last case. Would that make sense?
I think this trades one mess for a different one because then it's just a different bounds check and not all indexing is happening at ivar_idx anymore.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D76363/new/

https://reviews.llvm.org/D76363