[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:

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


More information about the llvm-dev mailing list