Re: [PING]: [PATCH]: Conditionally include target specific files while building TSAN

[Jakub Jelinek]

On Sat, Jan 24, 2015 at 01:23:22PM +0530, Venkataramanan Kumar wrote:
> I reused libgcc's "host_address" test and the patch passed normal
> bootstrap in x86_64.
> 
> Can you please check if this is fine ?

Can't you just use what configure.tgt already uses?

  x86_64-*-linux* | i?86-*-linux*)
        if test x$ac_cv_sizeof_void_p = x8; then
                TSAN_SUPPORTED=yes
                LSAN_SUPPORTED=yes
        fi
        ;;

Just make sure AC_CHECK_SIZEOF([void *]) is above this (seems it is).

So

TSAN_TARGET_DEPENDENT_OBJECTS=
case "${target}" in
  x86_64-*-linux* | i?86-*-linux*)
    if test x$ac_cv_sizeof_void_p = x8; then
      TSAN_TARGET_DEPENDENT_OBJECTS=tsan_rtl_amd64.lo
    fi;;
esac
AC_SUBST([TSAN_TARGET_DEPENDENT_OBJECTS])

?
Or even better move the TSAN_TARGET_DEPENDENT_OBJECTS initialization
to configure.tgt and just keep AC_SUBST([TSAN_TARGET_DEPENDENT_OBJECTS])
in configure.ac.

        Jakub