<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Tue, Oct 31, 2017 at 8:20 AM Pavel Labath <<a href="mailto:labath@google.com">labath@google.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 31 October 2017 at 15:12, Zachary Turner via lldb-commits<br>
<<a href="mailto:lldb-commits@lists.llvm.org" target="_blank">lldb-commits@lists.llvm.org</a>> wrote:<br>
> The takeaway from this example is nothing we don't already know.  We need<br>
> better test coverage.<br>
Actually, this was caught by a test (pretty much all of them), but<br>
only when building with libc++, as the code was "safe" with libstdc++<br>
due to the copy-on-write implementation of std::string (the temporary<br>
object shared storage with the longer-lived string it was copied<br>
from).<br></blockquote><div><br></div><div>What version of libstdc++?  AFAIK copy-on-write std::string is no longer conformant in C++11.</div><div><br></div><div>If this was really being masked due to a CoW string implementation, we need to make absolutely sure we are running tests on a bot with a conformant standard library. </div></div></div>