On Tue, 2012-11-20 at 13:00 +0400, Konstantin Serebryany wrote: > On Tue, Nov 20, 2012 at 12:19 PM, David Miller <da...@davemloft.net> wrote: > > From: Konstantin Serebryany <konstantin.s.serebry...@gmail.com> > > Date: Tue, 20 Nov 2012 11:41:03 +0400 > > > >> Ok. Will this work? > >> > >> // Are we using 32-bit or 64-bit syscalls? > >> // x32 (which defines __x86_64__) has __WORDSIZE == 32 > >> // but it still needs to use 64-bit syscalls. > >> #if defined(__x86_64__) || __WORDSIZE == 64 > >> # define SANITIZER_LINUX_USES_64BIT_SYSCALLS 1 > >> #else > >> # define SANITIZER_LINUX_USES_64BIT_SYSCALLS 0 > >> #endif > > > > Of course, as it matches the test H.J. Liu's patch used. > > > > Please, at a bare minimum, give him some credit for this fix. > > Done: http://llvm.org/viewvc/llvm-project?rev=168358&view=rev
I assume we are just waiting for someone to commit this to the GCC src, correct? David (Miller), were you going to do that? I'd like that change committed before I commit our ppc asan changes. Peter
