[PATCH] D34982: [Polly][WIP] Fully-Indexed static expansion
Bonfante Nicolas via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Jul 20 07:47:32 PDT 2017
niosega updated this revision to Diff 107505.
niosega marked 5 inline comments as done.
niosega added a comment.
In this revision, we have done :
- Use of c++ bindings for isl instead of direct isl
- Implementation of the reads expansion
- Add constraint for the write map so that the in dims are link to the out dims
- Remove useless JSCoP file
- Create a new expanded SAI for each statement
- Add a check method before doing the expansion to avoid partial expansion
Problems :
- Do I have to expand already fully-indexed array (for example, the first write of B in the test case "too-many-writes.ll" ) ?
For now, we can not expand :
- Scalar access
- MayWrite access
- SAI with more than one write
- SAI with read that can cause partial read access (because polly can not handle partial read access)
https://reviews.llvm.org/D34982
Files:
include/polly/LinkAllPasses.h
include/polly/ScopInfo.h
lib/CMakeLists.txt
lib/Support/RegisterPasses.cpp
lib/Transform/MaximalStaticExpansion.cpp
test/MaximalStaticExpansion/partial_access.ll
test/MaximalStaticExpansion/too_many_writes.ll
test/MaximalStaticExpansion/working_expansion.ll
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D34982.107505.patch
Type: text/x-patch
Size: 27728 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170720/050ead13/attachment.bin>
More information about the llvm-commits
mailing list