re: port-sparc/58352: ld: /usr/lib/libubsan.so: undefined reference to `__sync_val_compare_and_swap_8'

[matthew green <mrg%eterna23.net@localhost>]

The following reply was made to PR port-sparc/58352; it has been noted by GNATS.

From: matthew green <mrg%eterna23.net@localhost>
To: Rin Okuyama <rokuyama.rk%gmail.com@localhost>
Cc: gnats-bugs%netbsd.org@localhost, port-sparc-maintainer%netbsd.org@localhost,
    gnats-admin%netbsd.org@localhost, netbsd-bugs%netbsd.org@localhost,
    campbell+netbsd%mumble.net@localhost
Subject: re: port-sparc/58352: ld: /usr/lib/libubsan.so: undefined reference to `__sync_val_compare_and_swap_8'
Date: Thu, 20 Jun 2024 03:04:52 +1000

 > Yeah, this is just what I tried. It works at least on i386, but
 > diff is *really* large:
 >
 > https://gist.github.com/rokuyama/24d0ed7e537fffc07e2975c3880817f1
 
 i don't think this is that large.
 
 the only thing i'd really prefer would be to move the define
 for SANITIZER_NO_64BIT_ATOMIC_OPS from the Makefile into a common
 header such that you can define SANITIZER_STATE_T like you have.
 
 in the common header (not sure which one off hand), inside a
 SANITIZER_NETBSD section, check for our __HAVE_64BIT_ATOMIC_OPS
 or whatever it is, and define SANITIZER_NO_64BIT_ATOMIC_OPS to
 suit...
 
 
 i think this is do-able.  it's slightly more diffs than the
 current ones in gcc/dist/libsanitizer, but most of it is one
 bigger chunk, and all of them are fairly simple conversion
 (u64 -> SANITIZER_STATE_T), so keeping it in the future would
 not be too hard.  (it only adds about 4% to the total in all
 of gcc/dist.)
 
 thanks.
 
 
 .mrg.