[LLVMdev] Patches: Range insertion for DenseSet; definition of DenseMapInfo<char>

Chris Lattner clattner at apple.com
Wed Apr 1 12:51:30 PDT 2009


On Apr 1, 2009, at 11:54 AM, Kevin Fan wrote:
> Here are two minor patches.  The first adds an insert method to
> DenseSet that takes two iterators representing the beginning and
> ending of a range of items to insert.  I lifted the code verbatim from
> DenseMap.h.
> The second patch defines DenseMapInfo for chars.  This is useful
> because DenseSet is implemented as a DenseMap of (Type, char), so
> anyone who uses DenseSet has to define DenseMapInfo for char.  Let me
> know if this definition should go in some place other than DenseMap.h.

Looks great, applied:
http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20090330/075951.html

For future reference, we greatly appreciate patches sent in as  
attachments instead of inline, this makes it easier to apply.  Thanks  
for the patch!

-Chris



More information about the llvm-dev mailing list