tech-toolchain archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: libsanitizer per-arch defines



On 15.04.2019 17:44, coypu%sdf.org@localhost wrote:
> Why not make this but for every architecture?
> I needed RISCV.
> 
> It seems like it matches for a lot of them.
> 
> Index: external/gpl3/gcc/dist/libsanitizer/sanitizer_common/sanitizer_linux.cc
> ===================================================================
> RCS file: /cvsroot/src/external/gpl3/gcc/dist/libsanitizer/sanitizer_common/sanitizer_linux.cc,v
> retrieving revision 1.26
> diff -u -r1.26 sanitizer_linux.cc
> --- external/gpl3/gcc/dist/libsanitizer/sanitizer_common/sanitizer_linux.cc	5 Feb 2019 12:56:43 -0000	1.26
> +++ external/gpl3/gcc/dist/libsanitizer/sanitizer_common/sanitizer_linux.cc	15 Apr 2019 15:44:09 -0000
> @@ -1542,6 +1542,11 @@
>    *pc = _UC_MACHINE_PC(ucontext);
>    *sp = _UC_MACHINE_SP(ucontext);
>    *bp = ucontext->uc_mcontext.__gregs[1];	/* XXX */
> +#elif SANITIZER_NETBSD
> +  ucontext_t *ucontext = (ucontext_t*)context;
> +  *pc = _UC_MACHINE_PC(ucontext);
> +  *sp = _UC_MACHINE_SP(ucontext);
> +  *bp = _UC_MACHINE_FP(ucontext);
>  #elif defined(__s390__)
>    ucontext_t *ucontext = (ucontext_t*)context;
>  # if defined(__s390x__)
> 

This was already unified upstream and in previous downstream version of GCC.

https://github.com/gcc-mirror/gcc/blob/master/libsanitizer/sanitizer_common/sanitizer_linux.cc#L1880

We will get it from upstream together with GCC9.

Please unify it downstream again.

Attachment: signature.asc
Description: OpenPGP digital signature



Home | Main Index | Thread Index | Old Index