[patch] RFC asan support for i?86/x86_64-*freebsd*

Jeff Law law@redhat.com
Mon Nov 30 22:27:00 GMT 2015


On 11/29/2015 03:10 PM, Andreas Tobler wrote:
> All,
>
> this patch adds support for asan for i?86/x86_64-*freebsd*.
>
> Test results can be found on the list.
>
> These modifications belong only to gcc. There is one modification to
> asan/asan_linux.cc, this one is sent upstream. Until this one is in, my
> patch is on hold.
>
> One thing to note, FreeBSD does not need to link against -ldl. That is
> why I added an extra config check.
>
> But nevertheless I'd like to get some comments on the patch.
>
> Thanks to Jakub and Dan McGregor.
>
> Thanks,
> Andreas
>
>
> 2015-11-29  Andreas Tobler  <andreast@gcc.gnu.org>
>
>      * config/i386/i386.h: Define two new macros:
>      SUBTARGET_SHADOW_OFFSET_64 and SUBTARGET_SHADOW_OFFSET_32.
>      * config/i386/i386.c (ix86_asan_shadow_offset): Use these macros.
>      * config/i386/darwin.h: Override the SUBTARGET_SHADOW_OFFSET_64
>      macro.
>      * config/i386/freebsd.h: Override the SUBTARGET_SHADOW_OFFSET_64
>      and the SUBTARGET_SHADOW_OFFSET_32 macro.
>      * config/freebsd.h (LIBASAN_EARLY_SPEC): Define.
>      (LIBTSAN_EARLY_SPEC): Likewise.
>      (LIBLSAN_EARLY_SPEC): Likewise.
>
> 2015-11-29  Andreas Tobler  <andreast@gcc.gnu.org>
>
>      * configure.ac: Replace the hard-coded -ldl requirement for
>      link_sanitizer_common with a configure time check for -ldl.
>      * configure: Regenerate.
>      * configure.tgt: Add x86_64- and i?86-*-freebsd* targets.
The configury bits are fine.  Uros would own review on the x86 specific 
changes.

jeff



More information about the Gcc-patches mailing list