[cfe-dev] [LLVMdev] Bug in MapVector::erase ?

Duncan P. N. Exon Smith dexonsmith at apple.com
Tue Jul 15 11:28:34 PDT 2014


> On 2014-Jul-15, at 11:07, Reid Kleckner <rnk at google.com> wrote:
> 
> Can we explicitly delete the erase method or do something else to document the fact that it is unsupported?  It was added incidentally in r211350, even though it was added and removed by Doug back in r175538 / r175449.
> 

I'm happy with it deleted or fixed (see WIP patch that fixes it w/o
tests).  For now, I'll fix it, and then David (or someone else) can
migrate the code to `remove_if()`.

FWIW, when it's fixed, it doesn't have to be "unsupported" -- it's
just *slow*.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: mapvector.patch
Type: application/octet-stream
Size: 1999 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-dev/attachments/20140715/00ee4949/attachment.obj>


More information about the cfe-dev mailing list