[PATCH] D38916: [GlobalDCE] Use DenseMap instead of unordered_multimap for GVDependencies.

[Michael Zolotukhin via Phabricator via llvm-commits]

mzolotukhin added a comment.

Thanks!



================
Comment at: include/llvm/Transforms/IPO/GlobalDCE.h:38
   /// Global -> Global that uses this global.
-  std::unordered_multimap<GlobalValue *, GlobalValue *> GVDependencies;
+  DenseMap<GlobalValue *, SmallPtrSet<GlobalValue *, 4>> GVDependencies;
 
----------------
davide wrote:
> Why 4 ?
No particular reason. Any advice on a more scientific option to choose this value? :)


https://reviews.llvm.org/D38916