[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