<div dir="ltr">I'm fine with this going in for now, however, I don't think it answers any questions of "more than a workaround".<div><div><br><div><div class="gmail_quote"><div dir="ltr">On Thu, Jul 28, 2016 at 2:02 PM Nirav Davé <<a href="mailto:niravd@google.com">niravd@google.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">The fact that Hans still found the error after the r277027 patch is evidence that the second part of this patch is necessary as well (testing with Ismall showed that r277027 was necessary so both are needed). I think that indirectly answers the remaining question of whether copying or moving debug values is the right thing to do and we should be okay upstreaming this patch. </div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Jul 28, 2016 at 4:36 PM, Yaron Keren <span dir="ltr"><<a href="mailto:yaron.keren@gmail.com" target="_blank">yaron.keren@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">yaron.keren added a comment.<br>
<br>
PR28749 was created with trunk clang on Ubuntu x64.<br>
It reproduced reliably but is very fragile, changing just about anything in the code makes the problem disappear.<br>
<br>
<br>
<a href="https://reviews.llvm.org/D22858" rel="noreferrer" target="_blank">https://reviews.llvm.org/D22858</a><br>
<br>
<br>
<br>
</blockquote></div><br></div>
</blockquote></div></div></div></div></div>