This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Fix sanitizer build on sparc (PR sanitizer/59758)
- From: jemarch at gnu dot org (Jose E. Marchesi)
- To: Konstantin Serebryany <konstantin dot s dot serebryany at gmail dot com>
- Cc: Jakub Jelinek <jakub at redhat dot com>, "Jose E. Marchesi" <jose dot marchesi at oracle dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Wed, 19 Feb 2014 13:45:09 +0100
- Subject: Re: [PATCH] Fix sanitizer build on sparc (PR sanitizer/59758)
- Authentication-results: sourceware.org; auth=none
- References: <87ha7w2tlk dot fsf at oracle dot com> <20140218180049 dot GQ22862 at tucnak dot redhat dot com> <CAGQ9bdwYJDq_Cn_AHfSKmt1ewVTYP6vdm+EGXzT+=viNOS1fTg at mail dot gmail dot com>
> On Tue, Feb 18, 2014 at 06:55:51PM +0100, Jose E. Marchesi wrote:
> > This patch fixes builds with --enable-sanitizer, which seems to be the
> > default for sparc now.
> >
> > Build tested in a sparc64-*-linux-gnu system with linux 3.8.13 headers.
> >
> > 2014-02-18 Jose E. Marchesi <jose.marchesi@oracle.com>
> >
> > PR sanitizer/59758
> > * sanitizer_common/sanitizer_platform_limits_posix.h (__sanitizer):
> > Define struct__old_kernel_stat_sz, struct_kernel_stat_sz and
> > struct_kernel_stat64_sz for sparc targets (both 32 and 64 bits).
> > (__sanitizer_ipc_perm): Adjust for sparc targets.
> > (__sanitizer_shmid_ds): Likewise.
> > (__sanitizer_sigaction): Likewise.
> > (IOC_SIZE): Likewise.
> >
> > * sanitizer_common/sanitizer_platform_limits_linux.cc (time_t):
> > defined as __kernel_time_t, which is needed for sparc.
> > (struct___old_kernel_stat_sz): Don't check if __sparc__ is defined.
>
> Please talk to Konstantin about getting this into the upstream compiler-rt
> repository, we don't need to wait for a merge from there, so once it
> is accepted there, the same patch can be applied to gcc too.
Right. Please read
https://code.google.com/p/address-sanitizer/wiki/HowToContribute
Ok, I follow up on llvm-commits@cs.uiuc.edu then..