[PATCH] D24564: [LoopInterchange] Track all dependencies, not just anti dependencies.

Matthew Simpson via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 21 11:20:43 PDT 2016


mssimpso added inline comments.

================
Comment at: lib/Transforms/Scalar/LoopInterchange.cpp:120
@@ +119,3 @@
+        // Track Output, Flow, and Anti dependencies.
+        if (D->isOrdered()) {
+          DEBUG(StringRef DepType =
----------------
I'm not sure this check is needed. The unordered dependences are input dependences (RAR). It looks like the RAR case is explicitly checked a few lines up.


https://reviews.llvm.org/D24564





More information about the llvm-commits mailing list