[PATCH] D42151: [Polly][ScopBuilder] Make -polly-stmt-granularity=scalar-indep the default.

Michael Kruse via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 16 17:01:41 PST 2018


Meinersbur created this revision.
Meinersbur added reviewers: grosser, bollu.
Meinersbur added a project: Polly.

Splitting basic blocks into multiple statements if there are now additional scalar dependencies gives more freedom to the scheduler, but more statements also means higher compile-time complexity. Switch to finer statement granularity, the additional compile time should be limited by the number of operations quota.


Repository:
  rPLO Polly

https://reviews.llvm.org/D42151

Files:
  lib/Analysis/ScopBuilder.cpp
  test/lit.site.cfg.in


Index: test/lit.site.cfg.in
===================================================================
--- test/lit.site.cfg.in
+++ test/lit.site.cfg.in
@@ -59,6 +59,7 @@
                                  + ' -polly-use-llvm-names '
                                  + ' -polly-import-jscop-dir=%S '
                                  + ' -polly-codegen-verify '
+                                 + ' -polly-stmt-granularity=bb '
                                  ))
 
 if config.enable_gpgpu_codegen == 'TRUE' :
Index: lib/Analysis/ScopBuilder.cpp
===================================================================
--- lib/Analysis/ScopBuilder.cpp
+++ lib/Analysis/ScopBuilder.cpp
@@ -115,7 +115,8 @@
                           "Scalar independence heuristic"),
                clEnumValN(GranularityChoice::Stores, "store",
                           "Store-level granularity")),
-    cl::init(GranularityChoice::BasicBlocks), cl::cat(PollyCategory));
+    cl::init(GranularityChoice::ScalarIndependence), cl::cat(PollyCategory),
+    cl::ZeroOrMore);
 
 void ScopBuilder::buildPHIAccesses(ScopStmt *PHIStmt, PHINode *PHI,
                                    Region *NonAffineSubRegion,


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D42151.130068.patch
Type: text/x-patch
Size: 1182 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180117/0bc70ac8/attachment.bin>


More information about the llvm-commits mailing list