* [merged] tools-build-tweak-unused-value-workaround.patch removed from -mm tree
@ 2020-04-23 19:51 akpm
0 siblings, 0 replies; only message in thread
From: akpm @ 2020-04-23 19:51 UTC (permalink / raw)
To: gbiv, mm-commits, ndesaulniers
The patch titled
Subject: tools/build: tweak unused value workaround
has been removed from the -mm tree. Its filename was
tools-build-tweak-unused-value-workaround.patch
This patch was dropped because it was merged into mainline or a subsystem tree
------------------------------------------------------
From: George Burgess IV <gbiv@google.com>
Subject: 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/qqZ9r3
Link: http://lkml.kernel.org/r/20200414195638.156123-1-gbiv@google.com
Signed-off-by: George Burgess IV <gbiv@google.com>
Cc: Nick Desaulniers <ndesaulniers@google.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
---
tools/build/feature/test-sync-compare-and-swap.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
--- a/tools/build/feature/test-sync-compare-and-swap.c~tools-build-tweak-unused-value-workaround
+++ a/tools/build/feature/test-sync-compare-and-swap.c
@@ -7,7 +7,7 @@ int main(int argc, char *argv[])
{
uint64_t old, new = argc;
- argv = argv;
+ (void)argv;
do {
old = __sync_val_compare_and_swap(&x, 0, 0);
} while (!__sync_bool_compare_and_swap(&x, old, new));
_
Patches currently in -mm which might be from gbiv@google.com are
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2020-04-23 19:51 UTC | newest]
Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-04-23 19:51 [merged] tools-build-tweak-unused-value-workaround.patch removed from -mm tree akpm
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.