[llvm-commits] PATCH: Split the implementation of DenseMap to allow custom allocation strategies for the buckets array

Chandler Carruth chandlerc at google.com
Fri Jun 15 18:06:44 PDT 2012


On Fri, Jun 15, 2012 at 2:23 PM, Benjamin Kramer
<benny.kra at googlemail.com>wrote:

>
> On 15.06.2012, at 20:22, Chandler Carruth wrote:
>
> > The goal is to have a SmallDenseMap class that shares most of the logic
> (and iterators etc) with DenseMap, but allocates a small bucket array
> in-line.
> >
> > There are several smaller refactoring steps that clean things up after
> this one, but this is the invasive change I really want pre-commit review
> on. The rest should follow easily afterward.
>
> I dislike having two methods named "init" with different semantics,
> otherwise LGTM.
>

Done, and a few other tweaks discussed on IRC. Submitted as r158585, thanks
for the review!
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20120615/125ec82c/attachment.html>


More information about the llvm-commits mailing list