Revert libsanitizer patches or fix 59009
Peter Bergner
bergner@vnet.ibm.com
Fri Nov 15 14:59:00 GMT 2013
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
up the last of his HPPA build issues. I have confirmed on powerpc64-linux
that there are no libsanitizer testsuite regressions.
Is this ok for trunk?
I left out the mroute6.h change, as even without that being included,
I'm still seeing more related failures on SLES10/RHEL5 builds due to
struct ustat being multiply defined, so I thought we would leave that
change to be included with whatever else I need to get the build to
work on older systems.
Peter
PR sanitizer/59009
* sanitizer_common/sanitizer_platform_limits_posix.cc: Temporarily
ifdef out more interceptor source to fix HPPA build.
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);
More information about the Gcc-patches
mailing list