[PATCH] D21951: [LVI] Use DenseMap::find_as in LazyValueInfo.

Justin Lebar via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 1 21:15:39 PDT 2016


jlebar created this revision.
jlebar added a reviewer: reames.
jlebar added a subscriber: llvm-commits.

This lets us avoid creating and destroying a CallbackVH every time we
check the cache.

This is good for a 2% e2e speedup when compiling one of the large Eigen
tests at -O3.

FTR, I tried making the ValueCache hashtable one-level -- i.e., mapping
a pair (Value*, BasicBlock*) to a lattice value, and that didn't seem to
provide any additional improvement.  Saving a word in LVILatticeVal by
merging the Tag and Val fields also didn't yield a speedup.

http://reviews.llvm.org/D21951

Files:
  lib/Analysis/LazyValueInfo.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D21951.62599.patch
Type: text/x-patch
Size: 3128 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160702/b6e1ed72/attachment.bin>


More information about the llvm-commits mailing list