<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Sat, Aug 9, 2014 at 4:17 AM, Ivan A. Kosarev <span dir="ltr"><<a href="mailto:ikosarev@accesssoftek.com" target="_blank">ikosarev@accesssoftek.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hello Richard,<br>
<br>
As of today, the issue with system definitions on FreeBSD 9.2 in 32-bit mode addressed by this diff:<br>
<br>
<a href="http://reviews.llvm.org/D4623" target="_blank">http://reviews.llvm.org/D4623</a><br>
<br>
is the only one that prevent us from enabling running (and passing) 32-bit tests on the FreeBSD sanitizers buildbot.<br>
<br>
Can you please let me know if there is any decision about this patch? Do you think we should keep the tests disabled or apply the diff locally or it could be altered somehow to be more acceptable?</blockquote><div><br></div>
<div>I do not think it is our job to "fix" partially-implemented OS features. This patch would take us in a very troubling direction, and it's not clear that we would ever be able to remove this hack. Please try to find another way to get the result you desire. If the sanitizers + FreeBSD 9.2 needs some patch to be applied to the system headers, then that patch should be provided by FreeBSD or possibly by the sanitizers, and *not* by the compiler.</div>
</div></div></div>