On Sat, Aug 31, 2013 at 04:04:03PM +0100, Iain Sandoe wrote:
> Hi,
>
> On 30 Aug 2013, at 20:43, Jakub Jelinek wrote:
>
> > On Fri, Aug 30, 2013 at 09:38:01PM +0200, Dominique Dhumieres wrote:
> >>> I've just merged ubsan into trunk. Please send complaints my way.
> >>
> >> Bootstrap is broken on x86_64-apple-darwin10:
>
> > (wonder why not
> > libasan_la_LIBADD = $(top_builddir)/sanitizer_common/libsanitizer_common.la
> > if !USING_MAC_INTERPOSE
> > libasan_la_LIBADD += $(top_builddir)/interception/libinterception.la
> > endif
> > libasan_la_LIBADD += $(LIBSTDCXX_RAW_CXX_LDFLAGS)
> > ).
>
> … indeed, that's what I did for ubsan.
>
> > perhaps tsan/Makefile.am too (though, tsan isn't supported on darwin, so
> > it doesn't matter that much).
>
> tsan isn't relevant (yet): although, when we get some time to work on it,
> native thread support should be feasible for Darwin >= 11.
>
> ===
>
> the patch below fixes bootstrap - along the lines of your observation;
> it also fixes the specs to actually use the library (so that the tests pass
> too).
>
> bootstrapped x86_64-darwin12 for c,c++ and fortran (objc and ada bootstraps
> are broken from other causes).
> [also bootstrapped on x86_64-linux, and checked RUNTESTFLAGS="asan.exp
> ubsan.exp"]
>
> OK for trunk?
> Iain
I was just about to post something similar; thanks for the patch and sorry
for the breakage. Can't give you formal approval though.
> gcc:
> * config/darwin.h (LINK_COMMAND_SPEC_A): Revise sanitiser specs to
> include sanitise(undefined).
s/sanitise/sanitize/ ;)
Marek
