<div dir="ltr">I don't have any stake in non-trunk branches, so this is up to Hans. <br><div>The change looks pretty benign for a merge</div><div><br></div><div>--kcc </div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Feb 1, 2017 at 12:30 PM, Dimitry Andric <span dir="ltr"><<a href="mailto:dimitry@andric.com" target="_blank">dimitry@andric.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">This commit has now baked a few days, and I didn't see any further problems.  Kostya, Reid, is this OK to merge into release_40?<br>
<span class="HOEnZb"><font color="#888888"><br>
-Dimitry<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
On 30 Jan 2017, at 20:06, Dimitry Andric via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>> wrote:<br>
><br>
> Author: dim<br>
> Date: Mon Jan 30 13:06:13 2017<br>
> New Revision: 293536<br>
><br>
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=293536&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-<wbr>project?rev=293536&view=rev</a><br>
> Log:<br>
> Recommit: Stop intercepting some malloc-related functions on FreeBSD and<br>
> macOS<br>
><br>
> Summary:<br>
> In <a href="https://bugs.freebsd.org/215125" rel="noreferrer" target="_blank">https://bugs.freebsd.org/<wbr>215125</a> I was notified that some configure<br>
> scripts attempt to test for the Linux-specific `mallinfo` and `mallopt`<br>
> functions by compiling and linking small programs which references the<br>
> functions, and observing whether that results in errors.<br>
><br>
> FreeBSD and macOS do not have the `mallinfo` and `mallopt` functions, so<br>
> normally these tests would fail, but when sanitizers are enabled, they<br>
> incorrectly succeed, because the sanitizers define interceptors for<br>
> these functions.  This also applies to some other malloc-related<br>
> functions, such as `memalign`, `pvalloc` and `cfree`.<br>
><br>
> Fix this by not intercepting `mallinfo`, `mallopt`, `memalign`,<br>
> `pvalloc` and `cfree` for FreeBSD and macOS, in all sanitizers.<br>
><br>
> Also delete the non-functional `cfree` wrapper for Windows, to fix the<br>
> test cases on that platform.<br>
><br>
> Reviewers: emaste, kcc, rnk<br>
><br>
> Subscribers: timurrrr, eugenis, hans, joerg, llvm-commits, kubamracek<br>
><br>
> Differential Revision: <a href="https://reviews.llvm.org/D27654" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D27654</a><br>
><br>
> Added:<br>
>    compiler-rt/trunk/test/asan/<wbr>TestCases/malloc-no-intercept.<wbr>c<br>
> Modified:<br>
>    compiler-rt/trunk/lib/asan/<wbr>asan_malloc_linux.cc<br>
>    compiler-rt/trunk/lib/asan/<wbr>asan_malloc_win.cc<br>
>    compiler-rt/trunk/lib/lsan/<wbr>lsan_interceptors.cc<br>
>    compiler-rt/trunk/lib/<wbr>sanitizer_common/sanitizer_<wbr>platform_interceptors.h<br>
>    compiler-rt/trunk/lib/<wbr>sanitizer_common/tests/<wbr>sanitizer_allocator_testlib.cc<br>
<br>
</div></div></blockquote></div><br></div>