On Fri, Mar 04, 2016 at 07:03:09PM +0100, Marek Polacek wrote:
> Ok, version with dg-bogus:
>
> Bootstrapped/regtested on x86_64-linux, ok for trunk?
>
> 2016-03-04 Marek Polacek <[email protected]>
>
> PR c/69407
> * c-common.c (resolve_overloaded_builtin): Set TREE_USED for the fetch
> operations.
>
> * gcc.dg/atomic-op-6.c: New test.
> diff --git gcc/c-family/c-common.c gcc/c-family/c-common.c
> index 965cf49..25afa9c 100644
> --- gcc/c-family/c-common.c
> +++ gcc/c-family/c-common.c
> @@ -11443,6 +11443,10 @@ resolve_overloaded_builtin (location_t loc, tree
> function,
> && orig_code != BUILT_IN_ATOMIC_STORE_N)
> result = sync_resolve_return (first_param, result, orig_format);
>
> + if (fetch_op)
> + /* Prevent -Wunused-value warning. */
> + TREE_USED (result) = true;
> +
Can't sync_resolve_return return error_mark_node?
If it could, perhaps it would be saver to move this a few lines above the
sync_resolve_return call, where we've already checked that result is not
error_mark_node.
Otherwise LGTM.
Jakub
