<div dir="ltr">This should be fixed with <a href="https://reviews.llvm.org/rL294199">https://reviews.llvm.org/rL294199</a>.</div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Feb 6, 2017 at 12:25 PM, Vitaly Buka <span dir="ltr"><<a href="mailto:vitalybuka@google.com" target="_blank">vitalybuka@google.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="ltr">This patch breaks compiler-rt tests <a href="http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux/builds/801/steps/test%20standalone%20compiler-rt/logs/stdio" target="_blank">http://lab.llvm.org:8011/<wbr>builders/sanitizer-x86_64-<wbr>linux/builds/801/steps/test%<wbr>20standalone%20compiler-rt/<wbr>logs/stdio</a></div><br><div class="gmail_quote"><div><div class="h5"><div dir="ltr">On Mon, Feb 6, 2017 at 8:18 AM Kostya Kortchinsky via llvm-commits <<!--
--><a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>> wrote:<br></div></div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="h5"><div dir="ltr" class="m_7598803610270542090gmail_msg">Jonas, feel free to LGTM it to speed up the process :)</div><div class="gmail_extra m_7598803610270542090gmail_msg"><br class="m_7598803610270542090gmail_msg"><div class="gmail_quote m_7598803610270542090gmail_msg">On Mon, Feb 6, 2017 at 7:20 AM, Kostya Kortchinsky <span dir="ltr" class="m_7598803610270542090gmail_msg"><<a href="mailto:kostyak@google.com" class="m_7598803610270542090gmail_msg" target="_blank">kostyak@google.com</a>></span> wrote:<br class="m_7598803610270542090gmail_msg"><blockquote class="gmail_quote m_7598803610270542090gmail_msg" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><!--
--><div dir="ltr" class="m_7598803610270542090gmail_msg">Hi,<div class="m_7598803610270542090gmail_msg"><br class="m_7598803610270542090gmail_msg"><a href="https://reviews.llvm.org/D29527" class="m_7598803610270542090gmail_msg" target="_blank">https://reviews.llvm.org/<wbr>D29527</a> is out for review to fix this.<span class="m_7598803610270542090m_5508933356210100186HOEnZb m_7598803610270542090gmail_msg"><font color="#888888" class="m_7598803610270542090gmail_msg"><div class="m_7598803610270542090gmail_msg"><br class="m_7598803610270542090gmail_msg"></div><div class="m_7598803610270542090gmail_msg">Kostya</div></font></span></div></div><div class="m_7598803610270542090m_5508933356210100186HOEnZb m_7598803610270542090gmail_msg"><div class="m_7598803610270542090m_5508933356210100186h5 m_7598803610270542090gmail_msg"><div class="gmail_extra m_7598803610270542090gmail_msg"><br class="m_7598803610270542090gmail_msg"><!--
--><div class="gmail_quote m_7598803610270542090gmail_msg">On Mon, Feb 6, 2017 at 1:18 AM, Hahnfeld, Jonas <span dir="ltr" class="m_7598803610270542090gmail_msg"><<a href="mailto:Hahnfeld@itc.rwth-aachen.de" class="m_7598803610270542090gmail_msg" target="_blank">Hahnfeld@itc.rwth-aachen.de</a>></span> wrote:<br class="m_7598803610270542090gmail_msg"><blockquote class="gmail_quote m_7598803610270542090gmail_msg" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br class="m_7598803610270542090gmail_msg">
<div class="m_7598803610270542090gmail_msg"><div class="m_7598803610270542090m_5508933356210100186m_-9011879123930178650h5 m_7598803610270542090gmail_msg"><br class="m_7598803610270542090gmail_msg">
> -----Original Message-----<br class="m_7598803610270542090gmail_msg">
> From: llvm-commits [mailto:<a href="mailto:llvm-commits-bounces@lists.llvm.org" class="m_7598803610270542090gmail_msg" target="_blank">llvm-commits-bounces@<wbr>lists.llvm.org</a>] On Behalf<br class="m_7598803610270542090gmail_msg">
> Of Kostya Kortchinsky via llvm-commits<br class="m_7598803610270542090gmail_msg">
> Sent: Friday, February 03, 2017 9:50 PM<br class="m_7598803610270542090gmail_msg">
> To: <a href="mailto:llvm-commits@lists.llvm.org" class="m_7598803610270542090gmail_msg" target="_blank">llvm-commits@lists.llvm.org</a><br class="m_7598803610270542090gmail_msg">
> Subject: [compiler-rt] r294037 - [scudo] 32-bit quarantine sizes adjustments<br class="m_7598803610270542090gmail_msg">
> and bug fixes<br class="m_7598803610270542090gmail_msg">
><br class="m_7598803610270542090gmail_msg">
> Author: cryptoad<br class="m_7598803610270542090gmail_msg">
> Date: Fri Feb 3 14:49:42 2017<br class="m_7598803610270542090gmail_msg">
> New Revision: 294037<br class="m_7598803610270542090gmail_msg">
><br class="m_7598803610270542090gmail_msg">
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=294037&view=rev" rel="noreferrer" class="m_7598803610270542090gmail_msg" target="_blank">http://llvm.org/viewvc/llvm-<wbr>project?rev=294037&view=rev</a><br class="m_7598803610270542090gmail_msg">
> Log:<br class="m_7598803610270542090gmail_msg">
> [scudo] 32-bit quarantine sizes adjustments and bug fixes<br class="m_7598803610270542090gmail_msg">
><br class="m_7598803610270542090gmail_msg">
> Summary:<br class="m_7598803610270542090gmail_msg">
> The local and global quarantine sizes were not offering a distinction for<br class="m_7598803610270542090gmail_msg">
> 32-bit<br class="m_7598803610270542090gmail_msg">
> and 64-bit platforms. This is addressed with lower values for 32-bit.<br class="m_7598803610270542090gmail_msg">
><br class="m_7598803610270542090gmail_msg">
> When writing additional tests for the quarantine, it was discovered that<br class="m_7598803610270542090gmail_msg">
> when calling some of the allocator interface function prior to any<br class="m_7598803610270542090gmail_msg">
> allocation<br class="m_7598803610270542090gmail_msg">
> operation having occured, the test would crash due to the allocator not<br class="m_7598803610270542090gmail_msg">
> being<br class="m_7598803610270542090gmail_msg">
> initialized. This was addressed by making sure the allocator is initialized<br class="m_7598803610270542090gmail_msg">
> for<br class="m_7598803610270542090gmail_msg">
> those scenarios.<br class="m_7598803610270542090gmail_msg">
><br class="m_7598803610270542090gmail_msg">
> Relevant tests were added in interface.cpp and quarantine.cpp.<br class="m_7598803610270542090gmail_msg">
><br class="m_7598803610270542090gmail_msg">
> Last change being the removal of the extraneous link dependencies for the<br class="m_7598803610270542090gmail_msg">
> tests thanks to rL293220, anf the addition of the gc-sections linker flag.<br class="m_7598803610270542090gmail_msg">
><br class="m_7598803610270542090gmail_msg">
> Reviewers: kcc, alekseyshl<br class="m_7598803610270542090gmail_msg">
><br class="m_7598803610270542090gmail_msg">
> Reviewed By: alekseyshl<br class="m_7598803610270542090gmail_msg">
><br class="m_7598803610270542090gmail_msg">
> Subscribers: llvm-commits<br class="m_7598803610270542090gmail_msg">
><br class="m_7598803610270542090gmail_msg">
> Differential Revision: <a href="https://reviews.llvm.org/D29341" rel="noreferrer" class="m_7598803610270542090gmail_msg" target="_blank">https://reviews.llvm.org/<wbr>D29341</a><br class="m_7598803610270542090gmail_msg">
><br class="m_7598803610270542090gmail_msg">
> Modified:<br class="m_7598803610270542090gmail_msg">
> compiler-rt/trunk/lib/scudo/<wbr>scudo_allocator.cpp<br class="m_7598803610270542090gmail_msg">
> compiler-rt/trunk/lib/scudo/<wbr>scudo_flags.cpp<br class="m_7598803610270542090gmail_msg">
> compiler-rt/trunk/lib/scudo/<wbr>scudo_flags.inc<br class="m_7598803610270542090gmail_msg">
> compiler-rt/trunk/test/scudo/<wbr>alignment.cpp<br class="m_7598803610270542090gmail_msg">
> compiler-rt/trunk/test/scudo/<wbr>double-free.cpp<br class="m_7598803610270542090gmail_msg">
> compiler-rt/trunk/test/scudo/<wbr>interface.cpp<br class="m_7598803610270542090gmail_msg">
> compiler-rt/trunk/test/scudo/<wbr>lit.cfg<br class="m_7598803610270542090gmail_msg">
> compiler-rt/trunk/test/scudo/<wbr>malloc.cpp<br class="m_7598803610270542090gmail_msg">
> compiler-rt/trunk/test/scudo/<wbr>memalign.cpp<br class="m_7598803610270542090gmail_msg">
> compiler-rt/trunk/test/scudo/<wbr>mismatch.cpp<br class="m_7598803610270542090gmail_msg">
> compiler-rt/trunk/test/scudo/<wbr>options.cpp<br class="m_7598803610270542090gmail_msg">
> compiler-rt/trunk/test/scudo/<wbr>overflow.cpp<br class="m_7598803610270542090gmail_msg">
> compiler-rt/trunk/test/scudo/<wbr>preinit.cpp<br class="m_7598803610270542090gmail_msg">
> compiler-rt/trunk/test/scudo/<wbr>quarantine.cpp<br class="m_7598803610270542090gmail_msg">
> compiler-rt/trunk/test/scudo/<wbr>realloc.cpp<br class="m_7598803610270542090gmail_msg">
> compiler-rt/trunk/test/scudo/<wbr>secondary.cpp<br class="m_7598803610270542090gmail_msg">
> compiler-rt/trunk/test/scudo/<wbr>sized-delete.cpp<br class="m_7598803610270542090gmail_msg">
> compiler-rt/trunk/test/scudo/<wbr>sizes.cpp<br class="m_7598803610270542090gmail_msg">
><br class="m_7598803610270542090gmail_msg">
</div></div>> [...]<br class="m_7598803610270542090gmail_msg">
><br class="m_7598803610270542090gmail_msg">
> Modified: compiler-rt/trunk/test/scudo/<wbr>lit.cfg<br class="m_7598803610270542090gmail_msg">
> URL: <a href="http://llvm.org/viewvc/llvm-project/compiler-" rel="noreferrer" class="m_7598803610270542090gmail_msg" target="_blank">http://llvm.org/viewvc/llvm-<wbr>project/compiler-</a><br class="m_7598803610270542090gmail_msg">
> rt/trunk/test/scudo/lit.cfg?<wbr>rev=294037&r1=294036&r2=<wbr>294037&view=diff<br class="m_7598803610270542090gmail_msg">
> ==============================<wbr>============================<br class="m_7598803610270542090gmail_msg">
<span class="m_7598803610270542090gmail_msg">> ====================<br class="m_7598803610270542090gmail_msg">
> --- compiler-rt/trunk/test/scudo/<wbr>lit.cfg (original)<br class="m_7598803610270542090gmail_msg">
> +++ compiler-rt/trunk/test/scudo/<wbr>lit.cfg Fri Feb 3 14:49:42 2017<br class="m_7598803610270542090gmail_msg">
> @@ -19,12 +19,12 @@ config.suffixes = ['.c', '.cc', '.cpp']<br class="m_7598803610270542090gmail_msg">
> # C flags.<br class="m_7598803610270542090gmail_msg">
> c_flags = ([config.target_cflags] +<br class="m_7598803610270542090gmail_msg">
> ["-std=c++11",<br class="m_7598803610270542090gmail_msg">
> - "-lrt",<br class="m_7598803610270542090gmail_msg">
> - "-ldl",<br class="m_7598803610270542090gmail_msg">
<br class="m_7598803610270542090gmail_msg">
</span>With this change, I'm getting link errors for dlsym, shm_open and shm_unlink.<br class="m_7598803610270542090gmail_msg">
Can we read these libraries?<br class="m_7598803610270542090gmail_msg">
<br class="m_7598803610270542090gmail_msg">
Thanks,<br class="m_7598803610270542090gmail_msg">
Jonas<br class="m_7598803610270542090gmail_msg">
</blockquote></div><br class="m_7598803610270542090gmail_msg"></div>
</div></div></blockquote></div><br class="m_7598803610270542090gmail_msg"></div></div></div><span class="">
______________________________<wbr>_________________<br class="m_7598803610270542090gmail_msg">
llvm-commits mailing list<br class="m_7598803610270542090gmail_msg">
<a href="mailto:llvm-commits@lists.llvm.org" class="m_7598803610270542090gmail_msg" target="_blank">llvm-commits@lists.llvm.org</a><br class="m_7598803610270542090gmail_msg">
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" class="m_7598803610270542090gmail_msg" target="_blank">http://lists.llvm.org/cgi-bin/<wbr>mailman/listinfo/llvm-commits</a><br class="m_7598803610270542090gmail_msg">
</span></blockquote></div>
</blockquote></div><br></div>