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

Davide Italiano via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 17 15:17:35 PDT 2017


davide added a comment.

When it comes to this I generally try many of them until I find something reasonable. I'm pretty sure 4 is a reasonable value, but I was wondering whether you actually ran some tests on some different values before picking "the best" (for some definition of). Unlikely it's going to matter in any way, FWIW.


https://reviews.llvm.org/D38916





More information about the llvm-commits mailing list