[PATCH] D50925: [LICM] Hoist stores of invariant values to invariant addresses out of loops

Philip Reames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 21 20:30:12 PDT 2018


reames updated this revision to Diff 161883.
reames added a comment.

Address first couple comments, another update to come before re-review justified.

Max, I can't test the unique instruction bit through AST printer at the moment.  I could add printer support for the unique instruction helper, but I'm starting to seriously wonder if this belongs in AST at all.  If you don't mind, I'd prefer to exercise through LICM at the moment so splitting/moving the API is easy.


https://reviews.llvm.org/D50925

Files:
  include/llvm/Analysis/AliasSetTracker.h
  lib/Analysis/AliasSetTracker.cpp
  lib/Transforms/Scalar/LICM.cpp
  test/Transforms/LICM/atomics.ll
  test/Transforms/LICM/funclet.ll
  test/Transforms/LICM/promote-order.ll
  test/Transforms/LICM/store-hoisting.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D50925.161883.patch
Type: text/x-patch
Size: 7696 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180822/afe95ae4/attachment.bin>


More information about the llvm-commits mailing list