public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug c/78736] enum warnings in GCC (request for -Wenum-conversion to be added)
       [not found] <bug-78736-4@http.gcc.gnu.org/bugzilla/>
@ 2023-07-02  8:55 ` bp at alien8 dot de
  0 siblings, 0 replies; only message in thread
From: bp at alien8 dot de @ 2023-07-02  8:55 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78736

Boris <bp at alien8 dot de> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |bp at alien8 dot de

--- Comment #20 from Boris <bp at alien8 dot de> ---
(In reply to prathamesh3492 from comment #14)
> The patch enables warning with Wextra due to PR91593 and warnings with
> allmodconfig kernel build. Once these issues are resolved, we could
> consider promoting it to Wall.

Yes, you should. I just built 6.4 allmodconfig and there are no warnings
anymore likely because clang enables this warning by default and since we build
the kernel with both compilers, people have fixed them all in the meantime.

And clang did catch a legitimate error with this warning here while gcc didn't
using the default warning flags but it warned with -Wextra.

So I think we'll enable that warning by default in the kernel build and it does
make sense to do that in gcc too, I'd say.

Thx.

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2023-07-02  8:55 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <bug-78736-4@http.gcc.gnu.org/bugzilla/>
2023-07-02  8:55 ` [Bug c/78736] enum warnings in GCC (request for -Wenum-conversion to be added) bp at alien8 dot de

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).