Message ID | 1384525007.5562.23.camel@otta |
---|---|
State | New |
Headers | show |
On Fri, Nov 15, 2013 at 08:16:47AM -0600, Peter Bergner wrote: > On Wed, 2013-11-13 at 11:25 -0600, Peter Bergner wrote: > > On Wed, 2013-11-13 at 00:49 +0100, Jakub Jelinek wrote: > > > 2013-11-12 Jakub Jelinek <jakub@redhat.com> > > > > > > * sanitizer_common/sanitizer_platform_limits_linux.cc: Temporarily > > > ifdef out almost the whole source. > > > * sanitizer_common/sanitizer_common_syscalls.inc: Likewise. > > > > That helps, but as Pat reported in the bugzilla, it still is failing. > > With the following patch, we can now bootstrap on powerpc64-linux. > > > > Is this ok for trunk? > > > > Does this help the other architectures that are failing for the same > > build error? > > Ok, Dave reported in PR59009 that my last patch still left a few build > problems on HPPA. Dave tested the patch below and confirmed this cleans How can there be problems on HPPA? libsanitizer/configure.tgt says that hppa* is UNSUPPORTED, so libsanitizer should never be built there. Furthermore, it would be nice to understand why the sigaction is different. Jakub
On Fri, 2013-11-15 at 15:51 +0100, Jakub Jelinek wrote: > On Fri, Nov 15, 2013 at 08:16:47AM -0600, Peter Bergner wrote: > > Ok, Dave reported in PR59009 that my last patch still left a few build > > problems on HPPA. Dave tested the patch below and confirmed this cleans > > How can there be problems on HPPA? libsanitizer/configure.tgt says that > hppa* is UNSUPPORTED, so libsanitizer should never be built there. > Furthermore, it would be nice to understand why the sigaction is different. Well Dave's comment said he was building on hppa-linux: http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59009#c15 Dave, are you working on getting libsanitizer to work on hppa-linux and the patch just isn't upstream yet or??? Peter
Index: libsanitizer/sanitizer_common/sanitizer_platform_limits_posix.cc =================================================================== --- libsanitizer/sanitizer_common/sanitizer_platform_limits_posix.cc (revision 204841) +++ libsanitizer/sanitizer_common/sanitizer_platform_limits_posix.cc (working copy) @@ -846,6 +846,7 @@ CHECK_TYPE_SIZE(nfds_t); CHECK_TYPE_SIZE(sigset_t); +#ifdef SYSCALL_INTERCEPTION COMPILER_CHECK(sizeof(__sanitizer_sigaction) == sizeof(struct sigaction)); // Can't write checks for sa_handler and sa_sigaction due to them being // preprocessor macros. @@ -855,7 +856,6 @@ CHECK_STRUCT_SIZE_AND_OFFSET(sigaction, CHECK_STRUCT_SIZE_AND_OFFSET(sigaction, sa_restorer); #endif -#ifdef SYSCALL_INTERCEPTION #if SANITIZER_LINUX CHECK_TYPE_SIZE(__sysctl_args); CHECK_SIZE_AND_OFFSET(__sysctl_args, name);