From: Peter Bergner <berg...@vnet.ibm.com> Date: Tue, 20 Nov 2012 12:08:00 -0600
> 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. The problem is there are other dependencies in those two commits, the change doesn't apply cleanly. This situation is a very serious mess, and a merge burdon like this really shouldn't fall upon the gcc community at large.
