[PATCH] D18281: [SetVector] Add erase() method

[Daniel Berlin via llvm-commits]

Does it have the range definition?
(and what is our minimum version required at this point?)

On Fri, Mar 25, 2016 at 10:38 AM, Jun Bum Lim <junbuml at codeaurora.org>
wrote:

> junbuml added a comment.
>
> > /usr/include/c++/4.8/bits/vector.tcc:134:5: note: std::vector<_Tp,
> _Alloc>::iterator std::vector<_Tp, _Alloc>::erase(std::vector<_Tp,
> _Alloc>::iterator) [with _Tp = int; _Alloc = std::allocator<int>;
> std::vector<_Tp, _Alloc>::iterator = __gnu_cxx::__normal_iterator<int*,
> std::vector<int> >; typename std::_Vector_base<_Tp, _Alloc>::pointer = int*]
>
>
> From the Buildbot log above c++4.8 seems to be used which do not have
> c++11's definition of vector.erase(). Let me get the a non-const iterator
> as Danny mentioned.
>
>
> Repository:
>   rL LLVM
>
> http://reviews.llvm.org/D18281
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160325/37a9664e/attachment.html>