[PATCH] Fix char/short __sync_fetch_and_xxx (PR target/28924, take 2)

Roger Sayle roger@eyesopen.com
Fri Oct 6 16:02:00 GMT 2006


Hi Jakub,
On Fri, 6 Oct 2006, Jakub Jelinek wrote:
> Here is the updated patch;  I found that __sync_*_compare_and_swap
> and __sync_lock_test_and_set suffered from exactly the same problem,
> so I fixed that too.

Thanks.  Given this new smaller and safer patch is "obvious", I think
it's now OK for mainline in stage-3.  This should also allow Uros to
check-in his new sync-2.c test case to confirm his fix is correct.
I much prefer that we have appropriate test suite coverage for recent
changes in the tree, rather than fail to test adequately and identify
potential problems prior to a release.

Thanks to you both.

Roger
--



More information about the Gcc-patches mailing list