Commit a21151b9 authored by George Burgess IV's avatar George Burgess IV Committed by Linus Torvalds

tools/build: tweak unused value workaround

Clang has -Wself-assign enabled by default under -Wall, which always
gets -Werror'ed on this file, causing sync-compare-and-swap to be
disabled by default.

The generally-accepted way to spell "this value is intentionally
unused," is casting it to `void`.  This is accepted by both GCC and
Clang with -Wall enabled: https://godbolt.org/z/qqZ9r3Signed-off-by: default avatarGeorge Burgess IV <gbiv@google.com>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Cc: Nick Desaulniers <ndesaulniers@google.com>
Link: http://lkml.kernel.org/r/20200414195638.156123-1-gbiv@google.comSigned-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent 461e1565
...@@ -7,7 +7,7 @@ int main(int argc, char *argv[]) ...@@ -7,7 +7,7 @@ int main(int argc, char *argv[])
{ {
uint64_t old, new = argc; uint64_t old, new = argc;
argv = argv; (void)argv;
do { do {
old = __sync_val_compare_and_swap(&x, 0, 0); old = __sync_val_compare_and_swap(&x, 0, 0);
} while (!__sync_bool_compare_and_swap(&x, old, new)); } while (!__sync_bool_compare_and_swap(&x, old, new));
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment