Re: [PATCH] linux/compiler-clang.h: define HAVE_BUILTIN_BSWAP*
From: Luc Van Oostenryck
Date: Fri Feb 26 2021 - 02:48:30 EST
On Thu, Feb 25, 2021 at 05:45:09PM +0100, Arnd Bergmann wrote:
>
From: Arnd Bergmann <arnd@xxxxxxxx>
>
>
Fixes: 815f0ddb346c ("include/linux/compiler*.h: make compiler-*.h mutually exclusive")
>
Signed-off-by: Arnd Bergmann <arnd@xxxxxxxx>
>
---
>
include/linux/compiler-clang.h | 10 ++++++++++
>
1 file changed, 10 insertions(+)
>
>
diff --git a/include/linux/compiler-clang.h b/include/linux/compiler-clang.h
>
index 6478bff6fcc2..bbfa9ff6a2ec 100644
>
--- a/include/linux/compiler-clang.h
>
+++ b/include/linux/compiler-clang.h
>
@@ -33,6 +33,16 @@
>
#define __no_sanitize_thread
>
#endif
>
>
+/*
>
+ * sparse (__CHECKER__) pretends to be gcc, but can't do constant
>
+ * folding in __builtin_bswap*() (yet), so don't set these for it.
>
+ */
This is not true anymore since 2017. Also, a much recent version of
Sparse is needed for _Generic(), for example).
Can you remove the comment and the test for __CHECKER__?
Best regards,
-- Luc