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, Martin Husemann <martin%duskware.de@localhost>
Subject: re: port-sparc/58352: ld: /usr/lib/libubsan.so: undefined reference to `__sync_val_compare_and_swap_8'
Date: Sun, 30 Jun 2024 17:24:02 +1000

 Rin Okuyama writes:
 > After some investigation, I've found that sanitizer already
 > has its own 64-bit atomic ops for platforms without them.
 >
 > With the attached patch:
 > - libasan works on i386
 > - libubsan works on i386, powerpc, and vax (with WIP patches)
 >
 > Yes, this adds some performance penalty for these platforms,
 > however, it is better than having extra diff to upstream, IMO.
 
 nice!  this seems more maintainable, if slower, and i suspect
 32-bit platform users doing this are less common anyway.
 
 amused it is (a) in "sanitizer_atomic_clang.h", which is also
 used by GCC, and (b) includes "sanitizer_atomic_clang_mips.h",
 which has nothing mips- or clang- specific in it.
 
 thanks.
 
 
 .mrg.