Messages in this thread | | | Date | Fri, 26 Feb 2021 08:47:43 +0100 | From | Luc Van Oostenryck <> | Subject | Re: [PATCH] linux/compiler-clang.h: define HAVE_BUILTIN_BSWAP* |
| |
On Thu, Feb 25, 2021 at 05:45:09PM +0100, Arnd Bergmann wrote: > From: Arnd Bergmann <arnd@arndb.de> > > Fixes: 815f0ddb346c ("include/linux/compiler*.h: make compiler-*.h mutually exclusive") > Signed-off-by: Arnd Bergmann <arnd@arndb.de> > --- > 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
| |