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

Re: Bug#1028415: libfixmath: powerpc is in the UBSAN_ARCH list despite being unsupported


On 1/10/23 20:41, Rene Engelhard wrote:

Am 10.01.23 um 20:30 schrieb John Paul Adrian Glaubitz:
I just noticed that debian/rules adds the powerpc architecture to the UBSAN_ARCH
arch list despite powerpc not supporting `__sync_val_compare_and_swap_8 [1].

But libubsan.so exists, which was what was intended with this variable...

Removing it there would effectively disabling building and running the tests...

Yes, ubsan.so exists. But in some cases (i.e. some sanitizer checks) it requires
__sync_val_compare_and_swap_8 which is not available on 32-bit PowerPC.

However, __sync_val_compare_and_swap_8 is deprecated anyway and has been replaced
with __atomic_compare_exchange(), so I assume that the ubsan version in gcc-12 is

I think in LLVM, we actually replaced __sync_val_compare_and_swap with __atomic_
compare_exchange() at some point for this particular reasons.


[1] https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63368

 .''`.  John Paul Adrian Glaubitz
: :' :  Debian Developer
`. `'   Physicist
  `-    GPG: 62FF 8A75 84E0 2956 9546  0006 7426 3B37 F5B5 F913

Reply to: