<div dir="ltr">I can still see ubsan reports on lld: <div><a href="http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux-fast/builds/11343">http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux-fast/builds/11343</a><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Mar 23, 2016 at 5:20 AM, Rafael Espíndola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><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="HOEnZb"><div class="h5">
<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></div></blockquote></div><br></div>