[PATCH] D36647: [Polly][WIP] Scalar fully indexed expansion

Bonfante Nicolas via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 18 09:23:24 PDT 2017


niosega added inline comments.


================
Comment at: test/MaximalStaticExpansion/working_phi_expansion.ll:22
+;
+; CHECK: double MemRef_tmp_04__phi_Stmt_for_body_expanded[10000]; // Element size 8
+; CHECK: double MemRef_tmp_11__phi_Stmt_for_inc_expanded[10000][10000]; // Element size
----------------
This array should not be expanded because the expansion would lead to a read to the original array.
But the expanded memory accesses look like what is expected from @simbuerg analysis (in a previous comment). 

A check before expansion will be added in a next revision.


https://reviews.llvm.org/D36647





More information about the llvm-commits mailing list