[PATCH] D52312: [DenseMapInfo] Add implementation for SmallVector of pointers.

Duncan P. N. Exon Smith via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 20 15:29:53 PDT 2018


dexonsmith added a comment.

In https://reviews.llvm.org/D52312#1241110, @fhahn wrote:

> In https://reviews.llvm.org/D52312#1240764, @dexonsmith wrote:
>
> > I wonder, could these be replaced by `TinyPtrVector`?  They're only 8B.
>
>
> Interesting, I'll have a look. My only worry with this is that in the LSR and SLP use cases, it is very likely that the vector contains more than 1 element.


Sure, might be worth profiling compile time.


https://reviews.llvm.org/D52312





More information about the llvm-commits mailing list