[PATCH] D17381: [Polly] [RFC] Access level dependency analysis

Tobias Grosser via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 18 08:16:05 PST 2016


grosser added a comment.

Very nice start. Here a first comment:


================
Comment at: lib/Analysis/DependenceInfo.cpp:112
@@ -100,1 +111,3 @@
+      isl_space *space = isl_map_get_space(accdom);
+      accdom = tag(accdom, isl_space_copy(space));
 
----------------
You probably want to use MA->getID() to obtain a unique tag for each MemoryAccess in a statement. Even though these isl_ids are unique (due to the pointer they reference), they all are called "__polly_array_ref". We should probably give them a unique name too.


Repository:
  rL LLVM

http://reviews.llvm.org/D17381





More information about the llvm-commits mailing list