[PATCH] D17381: [Polly] [RFC] Memory reference level dependency analysis

Hongbin Zheng via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 18 08:46:50 PST 2016


etherzhhb added inline comments.

================
Comment at: lib/Analysis/DependenceInfo.cpp:126
@@ +125,3 @@
+      if (OptAnalysisLevel == REFERENCE_LEVEL_ANALYSIS)
+        accdom = tag(accdom, MA->getArrayId());
+
----------------
we can tag it with MA->getId() as well

================
Comment at: lib/Analysis/DependenceInfo.cpp:305-309
@@ -266,4 +304,7 @@
+
+  if (isl_union_map_is_empty(AccessSchedule) &&
+      OptAnalysisLevel == STATEMENT_LEVEL_ANALYSIS) {
     isl_union_map_free(AccessSchedule);
     Schedule = S.getScheduleTree();
   } else {
     auto *ScheduleMap =
----------------
Can we improve this if-else?


Repository:
  rL LLVM

http://reviews.llvm.org/D17381





More information about the llvm-commits mailing list