[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