[PATCH] D21041: [GVN] PRE can't hoist loads across calls in general.

[Eli Friedman via llvm-commits]

eli.friedman updated this revision to Diff 60478.
eli.friedman added a comment.

Improved control-flow check, fix for scalar PRE.

Still has crappy O(N^2) loop.  My current thinking is that it's possible to precompute it on a
per-block basis in GVN::runImpl.


http://reviews.llvm.org/D21041

Files:
  lib/Transforms/Scalar/GVN.cpp
  test/Transforms/GVN/local-pre.ll
  test/Transforms/GVN/pre-load.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D21041.60478.patch
Type: text/x-patch
Size: 5086 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160612/bfb85720/attachment.bin>