[PATCH] D111418: [GVN] Simple GVNHoist - limits

Momchil Velikov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 3 03:20:43 PDT 2021


chill updated this revision to Diff 384383.
chill edited the summary of this revision.
chill added a comment.

- Removed the maximum chain length limit as the current implementation does not iterate and does not need renumbering as the original GVNHoist
- Spent some effort to not separate certain instructions from their users (that's an outcome from some tuning/regression fixing with various benchmarks).


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D111418/new/

https://reviews.llvm.org/D111418

Files:
  llvm/lib/Transforms/Scalar/GVN.cpp
  llvm/test/Transforms/GVN/simple-gvn-hoist-limits.ll
  llvm/test/Transforms/PhaseOrdering/AArch64/hoisting-sinking-required-for-vectorization.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D111418.384383.patch
Type: text/x-patch
Size: 14727 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20211103/98e55bcc/attachment.bin>


More information about the llvm-commits mailing list