<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Mar 12, 2016 at 11:31 AM, Dimitry Andric via llvm-commits <span dir="ltr"><<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">dim abandoned this revision.<br>
dim added a comment.<br>
<br>
Okay, nevermind about this change.  In FreeBSD, I have simply imported <a href="http://reviews.llvm.org/rL255585" rel="noreferrer" target="_blank">http://reviews.llvm.org/rL255585</a> 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. `std::begin()`:<br>
<br>
  template <class _Cp><br>
  inline _LIBCPP_INLINE_VISIBILITY<br>
  typename _Cp::iterator<br>
  begin(_Cp& __c)<br>
  {<br>
      return __c.begin();<br>
  }<br>
<br>
E.g to be able to return anything, it needs to know the type.<br></blockquote><div><br></div><div>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.<br><br>Thanks for looking into/figuring out the issue!<br><br>- Dave</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="HOEnZb"><div class="h5"><br>
<br>
<a href="http://reviews.llvm.org/D18061" rel="noreferrer" target="_blank">http://reviews.llvm.org/D18061</a><br>
<br>
<br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</div></div></blockquote></div><br></div></div>