[PATCH] D33143: [Polly][Simplify] Remove identical write removal. NFC.

Tobias Grosser via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat May 13 02:58:04 PDT 2017


grosser added a comment.

Hi Michael,

it is not fully clear what this patch is about. I assume this is a WIP alternative to https://reviews.llvm.org/D33142. My current understanding is that https://reviews.llvm.org/D33142 is stable and this is still very much work in progress (also the patch in this submission seems broken). Not sure what you had in mind, but it seems we should go for https://reviews.llvm.org/D33142 and keep this discussing until until we figure out that they work well. Indeed, Indeed, I am worried about the O(N^3) complexity.



================
Comment at: overwritten.ll:3
+index 16fdbe11..d406134b 100644
+--- a/lib/Transform/Simplify.cpp
++++ b/lib/Transform/Simplify.cpp
----------------
Why is there a patch in this file?


https://reviews.llvm.org/D33143





More information about the llvm-commits mailing list