The patch for FreeBSD headers
Ivan A. Kosarev
ikosarev at accesssoftek.com
Mon Sep 1 05:34:21 PDT 2014
Thanks, Richard.
So, we state that the mainline version does not support 32-bit
sanitizers tests on 9.2. Now that the FreeBSD sanitizers buildbot has
been upgraded to v10, we will focus on supporting that version.
Thanks again.
On 08/25/2014 11:36 PM, Richard Smith wrote:
> On Sat, Aug 9, 2014 at 4:17 AM, Ivan A. Kosarev
> <ikosarev at accesssoftek.com <mailto:ikosarev at accesssoftek.com>> wrote:
>
> Hello Richard,
>
> As of today, the issue with system definitions on FreeBSD 9.2 in
> 32-bit mode addressed by this diff:
>
> http://reviews.llvm.org/D4623
>
> is the only one that prevent us from enabling running (and
> passing) 32-bit tests on the FreeBSD sanitizers buildbot.
>
> 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?
>
>
> 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.
--
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140901/c8a8f9cc/attachment.html>
More information about the llvm-commits
mailing list