[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