]> git.ipfire.org Git - thirdparty/gcc.git/commit
re PR middle-end/37908 (atomic NAND op generate wrong code; __sync_nand_and_fetch...
authorUros Bizjak <uros@gcc.gnu.org>
Mon, 17 Nov 2008 11:19:06 +0000 (12:19 +0100)
committerUros Bizjak <uros@gcc.gnu.org>
Mon, 17 Nov 2008 11:19:06 +0000 (12:19 +0100)
commit23462d4d8fb5b5e40d6a62dd285301228f13a37b
treeec588937bd71d8c137cd8e58e26176a069ebf17b
parent73ce7fcb8cdacf40e7342834e1af9d930e012e02
re PR middle-end/37908 (atomic NAND op generate wrong code; __sync_nand_and_fetch, __sync_fetch_and_nand)

PR middle-end/37908
* optabs.c (expand_sync_operation): Properly handle NAND case
by calculating ~(t1 & val) instead of (~t1 & val).
* builtins.c (expand_builtin_sync_operation): Warn for changed
semantics in NAND builtins.
* c.opt (Wsync-nand): New warning option.  Describe -Wsync-nand.

* doc/invoke.texi (Warning options): Add Wsync-nand.
* doc/extend.texi (Atomic Builtins) [__sync_fetch_and_nand]: Correct
__sync_fetch_and_nand builtin operation in the example.  Add a note
about changed semantics in GCC 4.4.
[__sync_nand_and_fetch]: Correct __sync_nand_and_fetch builtin
operation in the example.  Add a note about changed semantics in
GCC 4.4.

testsuite/ChangeLog:

PR middle-end/37908
* gcc.dg/pr37908.c: New test.
* gcc.dg/ia64-sync-1.c: Correct __sync_fetch_and_nand and
__sync_nand_and_fetch results.  Add dg-message to look for the warning
about changed semantics of NAND builtin.
(init_si, init_di): Change init value for __sync_fetch_and_nand to -1.
(test_si, test_di): Change expected result of
__sync_nand_and_fetch to ~7.
* gcc.dg/ia64-sync-2.c: Correct __sync_fetch_and_nand and
__sync_nand_and_fetch results.  Add dg-message to look for the warning
about changed semantics of NAND builtin.
(init_noret_si, init_noret_di): Change init value for
__sync_fetch_and_nand to -1.
(init_noret_si, init_noret_di): Change expected result of
__sync_nand_and_fetch to ~7.
* gcc.dg/sync-2.c: Correct __sync_fetch_and_nand and
__sync_nand_and_fetch results.  Add dg-message to look for the warning
about changed semantics of NAND builtin.
(init_qi, init_qi): Change init value for __sync_fetch_and_nand to -1.
(init_hi, init_hi): Change expected result of
__sync_nand_and_fetch to ~7.
* gcc.dg/sync-3.c: Copy from sync-2.c instead of including
the c source file.
* gcc.c-torture/compile/sync-1.c: Add dg-message to look for the
warning about changed semantics of NAND builtin.
* gcc.c-torture/compile/sync-2.c: Ditto.
* gcc.c-torture/compile/sync-3.c: Ditto.

From-SVN: r141942
15 files changed:
gcc/ChangeLog
gcc/builtins.c
gcc/c.opt
gcc/doc/extend.texi
gcc/doc/invoke.texi
gcc/optabs.c
gcc/testsuite/ChangeLog
gcc/testsuite/gcc.c-torture/compile/sync-1.c
gcc/testsuite/gcc.c-torture/compile/sync-2.c
gcc/testsuite/gcc.c-torture/compile/sync-3.c
gcc/testsuite/gcc.dg/ia64-sync-1.c
gcc/testsuite/gcc.dg/ia64-sync-2.c
gcc/testsuite/gcc.dg/pr37908.c [new file with mode: 0644]
gcc/testsuite/gcc.dg/sync-2.c
gcc/testsuite/gcc.dg/sync-3.c