[PATCH] D90640: [LoopFlatten] Widen the IV

Dave Green via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 13 00:45:07 PST 2020


dmgreen added a comment.

> now using RecursivelyDeleteDeadPHINode at the end to clean up dead phis. Doing this earlier proved to be a bit tricky with a few things in flight, but at least a nice clean up at the end.

What goes wrong when these are done earlier? I was hoping that it would help remove the need for OrigPHIs and Wide2OrigPHIs, cleaning up a bit of the awkwardness they are needed for.



================
Comment at: llvm/test/Transforms/LoopFlatten/widen-iv.ll:6
+; Function Attrs: nounwind
+define dso_local void @foo(i32* %A, i32 %N, i32 %M) local_unnamed_addr #0 {
+; CHECK-LABEL: @foo(
----------------
dso_local and local_unnamed_addr #0 can be removed.


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

https://reviews.llvm.org/D90640



More information about the llvm-commits mailing list