<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Mar 10, 2016 at 2:00 PM, 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 added a subscriber: EricWF.<br>
dim added a comment.<br>
<span class=""><br>
In <a href="http://reviews.llvm.org/D18061#372401" rel="noreferrer" target="_blank">http://reviews.llvm.org/D18061#372401</a>, @dblaikie wrote:<br>
<br>
> Is this a bug in libstdc++ - should it be requiring/relying on the operand<br>
>  to have an ::iterator typedef?<br>
<br>
<br>
</span>This compilation isn't using libstdc++, but libc++.  So if it's a bug in the C++ library, it's something in libc++...  However, I don't think so, really.  @EricWF, any opinions about this?<br></blockquote><div><br></div><div>Ah, curious. Then this wouldn't be an issue with GCC version. I would expect this to be evident with any compiler using that libc++?</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class=""><br>
> (& even if it is a bug, perhaps we should still workaround it - but I'm not<br>
<br>
>  entirely sure what the right workaround is, perhaps we should just have our<br>
<br>
>  own begin/end that don't have this feature/bug)<br>
<br>
<br>
</span>I'm not sure why reverse() doesn't just call .begin() and .end() on the argument.  There might have been a specific reason to use std::begin() and std::end().<br></blockquote><div><br></div><div>The most obvious is that it would work with arrays, ideally - but there are other containers that overload non-member begin/end to allow for iteration as well. Though I'm not sure if I tested that, etc.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
<br>
<a href="http://reviews.llvm.org/D18061" rel="noreferrer" target="_blank">http://reviews.llvm.org/D18061</a><br>
<div class="HOEnZb"><div class="h5"><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>