On Tue, Nov 13, 2012 at 02:17:55PM +0100, Dodji Seketeli wrote:
> What do the maintainers think?
Yes. And it shouldn't be just based on target CPU, but also based
on target OS, I don't think libsanitizer supports anything but linux (glibc
+ maybe android) right now, with some smaller or bigger tweaks it could
support darwin (but see the reports that it doesn't build there right now)
or mingw/cygwin? (but there is a PR that it doesn't build there).
So IMHO it should be a whitelist of supported targets with *) case
adding noconfigdirs="$noconfigdirs target-libsanitizer", rather than
blacklist of few unsupported ones. Can you please prepare a patch?
> > --- configure.ac (revision 193455)
> > +++ configure.ac (working copy)
> > @@ -547,6 +547,13 @@ case "${target}" in
> > ;;
> > esac
> >
> > +# Disable libsanitizer for some systems.
> > +case "${target}" in
> > + cris-*-* | crisv32-*-* | mmix-*-*)
> > + noconfigdirs="$noconfigdirs target-libsanitizer"
> > + ;;
> > +esac
> > +
Jakub
