Re: [PATCH] Add anitizer_linux_x86_64.lo if __x86_64__ is defined by $CC

2017-10-04 Thread Jakub Jelinek
On Sat, Sep 30, 2017 at 06:12:39PM -0700, H.J. Lu wrote: > Since size of "void *" is 4 bytes for x32, check if __x86_64__ is defined > by $CC, instead of > > if test x$ac_cv_sizeof_void_p = x8; then > > to decide wether anitizer_linux_x86_64.lo should be used. > > I am testing this on i686 and x

PING: [libsanitizer, PATCH] Add anitizer_linux_x86_64.lo if __x86_64__ is defined by $CC

2017-10-04 Thread H.J. Lu
On 9/30/17, H.J. Lu wrote: > Since size of "void *" is 4 bytes for x32, check if __x86_64__ is defined > by $CC, instead of > > if test x$ac_cv_sizeof_void_p = x8; then > > to decide wether anitizer_linux_x86_64.lo should be used. > > I am testing this on i686 and x86-64. OK for trunk and GCC 7 b

[PATCH] Add anitizer_linux_x86_64.lo if __x86_64__ is defined by $CC

2017-09-30 Thread H.J. Lu
Since size of "void *" is 4 bytes for x32, check if __x86_64__ is defined by $CC, instead of if test x$ac_cv_sizeof_void_p = x8; then to decide wether anitizer_linux_x86_64.lo should be used. I am testing this on i686 and x86-64. OK for trunk and GCC 7 branch if there are no regression? Please