[polly] r294831 - [NFC] Fix style issues of lib/Transform/ScheduleOptimizer.cpp.

Roman Gareev via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 10 23:14:37 PST 2017


Author: romangareev
Date: Sat Feb 11 01:14:37 2017
New Revision: 294831

URL: http://llvm.org/viewvc/llvm-project?rev=294831&view=rev
Log:
[NFC] Fix style issues of lib/Transform/ScheduleOptimizer.cpp.

Modified:
    polly/trunk/lib/Transform/ScheduleOptimizer.cpp

Modified: polly/trunk/lib/Transform/ScheduleOptimizer.cpp
URL: http://llvm.org/viewvc/llvm-project/polly/trunk/lib/Transform/ScheduleOptimizer.cpp?rev=294831&r1=294830&r2=294831&view=diff
==============================================================================
--- polly/trunk/lib/Transform/ScheduleOptimizer.cpp (original)
+++ polly/trunk/lib/Transform/ScheduleOptimizer.cpp Sat Feb 11 01:14:37 2017
@@ -265,7 +265,7 @@ getIsolateOptions(__isl_take isl_set *Is
 /// It may help to reduce the size of generated code.
 ///
 /// @param Ctx An isl_ctx, which is used to create the isl_union_set.
-static __isl_give isl_union_set *getAtomicOptions(__isl_take isl_ctx *Ctx) {
+static __isl_give isl_union_set *getAtomicOptions(isl_ctx *Ctx) {
   auto *Space = isl_space_set_alloc(Ctx, 0, 1);
   auto *AtomicOption = isl_set_universe(Space);
   auto *Id = isl_id_alloc(Ctx, "atomic", nullptr);
@@ -766,27 +766,24 @@ static bool containsOnlyMatMulDep(__isl_
   }
   isl_union_map_free(WAR);
   auto *RAW = D->getDependences(Dependences::TYPE_RAW);
-  auto *Domain = isl_map_domain(isl_map_copy(Schedule));
-  auto *Space = isl_space_map_from_domain_and_range(isl_set_get_space(Domain),
-                                                    isl_set_get_space(Domain));
-  isl_set_free(Domain);
+  auto *DomainSpace = isl_space_domain(isl_map_get_space(Schedule));
+  auto *Space = isl_space_map_from_domain_and_range(isl_space_copy(DomainSpace),
+                                                    DomainSpace);
   auto *Deltas = isl_map_deltas(isl_union_map_extract_map(RAW, Space));
   int DeltasDimNum = isl_set_dim(Deltas, isl_dim_set);
+  isl_set_free(Deltas);
   for (int i = 0; i < DeltasDimNum; i++) {
     auto *Val = isl_set_plain_get_val_if_fixed(Deltas, isl_dim_set, i);
-    if (Pos < 0 && isl_val_is_one(Val))
-      Pos = i;
+    Pos = Pos < 0 && isl_val_is_one(Val) ? i : Pos;
     if (isl_val_is_nan(Val) ||
         !(isl_val_is_zero(Val) || (i == Pos && isl_val_is_one(Val)))) {
       isl_val_free(Val);
       isl_union_map_free(RAW);
-      isl_set_free(Deltas);
       return false;
     }
     isl_val_free(Val);
   }
   isl_union_map_free(RAW);
-  isl_set_free(Deltas);
   return true;
 }
 




More information about the llvm-commits mailing list