[PATCH] D37424: [ForwardOp] Remove read accesses for all instructions that have been moved
Tobias Grosser via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sun Sep 3 06:29:08 PDT 2017
grosser created this revision.
grosser added a project: Polly.
Before this change only the memory accesses belonging to the roots of the
operand trees were removed, this resulted in some redundant reads, but more
importantly also caused crashes in the code generation, as instructions for
which a read access still existed where modeled as inter-statement use, which
caused polly-simplify's mark-and-sweep algorithm to not reach these statements,
which resulted in dropped instructions and consequently broken code.
https://reviews.llvm.org/D37424
Files:
lib/Transform/ForwardOpTree.cpp
test/ForwardOpTree/forward_load_tripleuse.ll
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D37424.113691.patch
Type: text/x-patch
Size: 8947 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170903/a22e0a9c/attachment.bin>
More information about the llvm-commits
mailing list