[Bug jit/96067] __atomic_compare_exchange_n should return bool instead of void
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96067 --- Comment #5 from CVS Commits --- The master branch has been updated by Antoni Boucher : https://gcc.gnu.org/g:611fdb0fc5b95ee15215e2e3679834f311919096 commit r12-5912-g611fdb0fc5b95ee15215e2e3679834f311919096 Author: Antoni Boucher Date: Sat Dec 11 16:42:54 2021 -0500 libgccjit: Add support for types used by atomic builtins [PR96066] [PR96067] 2021-12-11 Antoni Boucher gcc/jit/ PR target/96066 PR target/96067 * jit-builtins.c: Implement missing types for builtins. * jit-recording.c:: Allow sending a volatile const void * as argument. * jit-recording.h: New functions (is_volatile, is_const) and allow comparing qualified types. gcc/testsuite/ PR target/96066 PR target/96067 * jit.dg/all-non-failing-tests.h: Add test-builtin-types.c. * jit.dg/test-builtin-types.c * jit.dg/test-error-bad-assignment.c * jit.dg/test-fuzzer.c: Add fuzzing for type qualifiers. Signed-off-by: Antoni Boucher
[Bug jit/96067] __atomic_compare_exchange_n should return bool instead of void
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96067 Antoni changed: What|Removed |Added Resolution|--- |DUPLICATE Status|ASSIGNED|RESOLVED --- Comment #4 from Antoni --- Closing in favor of 96066. That should be using a number instead of `n` as in `__atomic_compare_exchange_4`. *** This bug has been marked as a duplicate of bug 96066 ***
[Bug jit/96067] __atomic_compare_exchange_n should return bool instead of void
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96067 Richard Biener changed: What|Removed |Added Status|NEW |ASSIGNED
[Bug jit/96067] __atomic_compare_exchange_n should return bool instead of void
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96067 Jonathan Wakely changed: What|Removed |Added Status|WAITING |NEW --- Comment #3 from Jonathan Wakely --- Thanks
[Bug jit/96067] __atomic_compare_exchange_n should return bool instead of void
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96067 --- Comment #2 from Antoni --- Created attachment 48835 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=48835=edit Reproducer for thebug Here's a reproducer for the bug. The doc says it should return bool (https://gcc.gnu.org/onlinedocs/gcc/_005f_005fatomic-Builtins.html), but this reproducer shows the following error: libgccjit.so: error: gcc_jit_block_end_with_return: mismatching types: return of __atomic_compare_exchange_n ((), (), (int)0, (bool)0, (int)0, (int)0) (type: void) in function hello (return type: bool)
[Bug jit/96067] __atomic_compare_exchange_n should return bool instead of void
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96067 Jonathan Wakely changed: What|Removed |Added Last reconfirmed||2020-07-06 Status|UNCONFIRMED |WAITING Ever confirmed|0 |1 --- Comment #1 from Jonathan Wakely --- (In reply to Antoni from comment #0) > The builtin __atomic_compare_exchange_n returns a void instead of bool. This is not a very useful bug report. Do you have a testcase? Is this something specific to libgccjit?