[PATCH] D18061: Add iterator types to iterator_range, to appease newer gcc's
David Blaikie via llvm-commits
llvm-commits at lists.llvm.org
Sat Mar 12 14:07:20 PST 2016
On Sat, Mar 12, 2016 at 11:31 AM, Dimitry Andric via llvm-commits <
llvm-commits at lists.llvm.org> wrote:
> dim abandoned this revision.
> dim added a comment.
> Okay, nevermind about this change. In FreeBSD, I have simply imported
> http://reviews.llvm.org/rL255585 for now (a full libc++ import will
> follow later). Gcc wanted to see `iterator` and `const_iterator` types in
> `iterator_range`, when `_LIBCPP_HAS_NO_TRAILING_RETURN` is defined, because
> libc++'s <iterator> then uses the following kind of definitions for e.g.
> template <class _Cp>
> inline _LIBCPP_INLINE_VISIBILITY
> typename _Cp::iterator
> begin(_Cp& __c)
> return __c.begin();
> E.g to be able to return anything, it needs to know the type.
Yeah, I'm pretty sure we require trailing return type for LLVM (I think we
have several explicit uses of it in the LLVM codebase) so it would be weird
for us to workaround non-standard features of the stdlib implementation
that only crop up only to work around non-conforming compilers that
wouldn't compile the rest of LLVM anyway.
Thanks for looking into/figuring out the issue!
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the llvm-commits