[PATCH] D13487: [Polly] Load/Store scalar accesses before/after the statement itself

Johannes Doerfert via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 6 23:28:20 PDT 2015


jdoerfert added inline comments.

================
Comment at: lib/CodeGen/BlockGenerators.cpp:1176
@@ -1166,1 +1175,3 @@
+    // copied directly.
+    if (MA->isPHI() && !IsPHIWrite)
       continue;
----------------
This part and the one below seem tricky and I have to understand the change better to get why/what happens here.

The idea of a single value map seems reasonable though. Are these two changes coupled?


http://reviews.llvm.org/D13487





More information about the llvm-commits mailing list