<div dir="ltr">Sweet! <div>r264239 should enable the failure mode, let's watch the bot. <br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Mar 23, 2016 at 7:20 PM, Pete Cooper <span dir="ltr"><<a href="mailto:peter_cooper@apple.com" target="_blank">peter_cooper@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="auto"><div>Thanks for suggesting we enable error reporting. I was hoping we could but didn't know how to go about doing it.</div><div><br></div><div>BTW, as of this build we are now also ubsan clean. <a href="http://lab.llvm.org" target="_blank">lab.llvm.org</a>:8011/builders/sanitizer-x86_64-linux-fast/builds/11344</div><div><br></div><div>Cheers</div><div>Pete<br><br>Sent from my iPhone</div><div><div class="h5"><div><br>On Mar 23, 2016, at 5:20 AM, Rafael Espíndola <<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>> wrote:<br><br></div><blockquote type="cite"><div><p dir="ltr">Kostya,</p>
<p dir="ltr">Can you change the bot to start reporting errors?</p>
<p dir="ltr">Thanks,<br>
Rafael</p>
<div class="gmail_quote">On Mar 22, 2016 6:48 PM, "Pete Cooper" <<a href="mailto:peter_cooper@apple.com" target="_blank">peter_cooper@apple.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi all<br>
<br>
Just to note that this has (finally!) fixed the last asan failure in the mach-o lld code.  The first passing results are <a href="http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux-fast/builds/11308" rel="noreferrer" target="_blank">http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux-fast/builds/11308</a><br>
<br>
We still have to fix the ubsan failures which look to be the same issue over and over again, and i’ll work on that next.<br>
<br>
Cheers,<br>
Pete<br>
<br>
PS, sorry this took so long.  I really hoped to have something in place at the end of last week but it took longer than expected to get my head around the YAML parser and its interaction with owning pointers.<br>
> On Mar 22, 2016, at 2:08 PM, Pete Cooper via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>> wrote:<br>
><br>
> Author: pete<br>
> Date: Tue Mar 22 16:08:39 2016<br>
> New Revision: 264097<br>
><br>
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=264097&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project?rev=264097&view=rev</a><br>
> Log:<br>
> Fix operator= on OwningAtomPtr to call destructor when needed.<br>
><br>
> If the LHS of 'a = b' already had an atom in it then we wouldn't<br>
> call the destructor.  This happens when we use something like<br>
> std::remove_if which is done in the CompactUnwindPass.  Should fix<br>
> the leaks on the mach-o/unwind-info-simple-x86_64.yaml test case.<br>
><br>
> Lang and I are going to take a look at removing OwningAtomPtr in<br>
> favour of a std::unique_ptr but just trying to get the bots green<br>
> so we have a good baseline first.<br>
><br>
> Modified:<br>
>    lld/trunk/include/lld/Core/Atom.h<br>
><br>
> Modified: lld/trunk/include/lld/Core/Atom.h<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/lld/trunk/include/lld/Core/Atom.h?rev=264097&r1=264096&r2=264097&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/lld/trunk/include/lld/Core/Atom.h?rev=264097&r1=264096&r2=264097&view=diff</a><br>
> ==============================================================================<br>
> --- lld/trunk/include/lld/Core/Atom.h (original)<br>
> +++ lld/trunk/include/lld/Core/Atom.h Tue Mar 22 16:08:39 2016<br>
> @@ -100,6 +100,8 @@ public:<br>
>   }<br>
><br>
>   void operator=(OwningAtomPtr&& ptr) {<br>
> +    if (atom)<br>
> +      runDestructor(atom);<br>
>     atom = ptr.atom;<br>
>     ptr.atom = nullptr;<br>
>   }<br>
><br>
><br>
> _______________________________________________<br>
> llvm-commits mailing list<br>
> <a href="mailto:llvm-commits@lists.llvm.org" target="_blank">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>
<br>
</blockquote></div>
</div></blockquote></div></div></div></blockquote></div><br></div>