https://gcc.gnu.org/bugzilla/show_bug.cgi?id=53131
Marek Polacek mpolacek at gcc dot gnu.org changed:
What|Removed |Added
Status|UNCONFIRMED |RESOLVED
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=53131
--- Comment #8 from Manuel López-Ibáñez manu at gcc dot gnu.org ---
(In reply to Manuel López-Ibáñez from comment #7)
In fact, the main show-stopper for adding -Wlogical-op to -Wextra (or -Wall)
is PR40172, which was the reason it was moved out
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=53131
Bug 53131 depends on bug 43772, which changed state.
Bug 43772 Summary: Errant -Wlogical-op warning when testing limits
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=43772
What|Removed |Added
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53131
--- Comment #7 from Manuel López-Ibáñez manu at gcc dot gnu.org 2012-05-07
13:42:10 UTC ---
In fact, the main show-stopper for adding -Wlogical-op to -Wextra (or -Wall) is
PR40172, which was the reason it was moved out of -Wextra in the first
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53131
--- Comment #2 from dcb dcb314 at hotmail dot com 2012-04-28 07:29:17 UTC ---
(In reply to comment #1)
(In reply to comment #0)
I am somewhat surprised that -Wlogical-op isn't part of either -Wall
or possibly -Wextra.
It could be in
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53131
--- Comment #3 from Manuel López-Ibáñez manu at gcc dot gnu.org 2012-04-28
10:37:04 UTC ---
(In reply to comment #2)
Do the warnings indicate bugs or not?
Yes. I checked the first ten.
And what do the kernel people say? Do they want
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53131
Marc Glisse marc.glisse at normalesup dot org changed:
What|Removed |Added
CC|
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53131
--- Comment #5 from dcb dcb314 at hotmail dot com 2012-04-28 12:17:57 UTC ---
(In reply to comment #4)
Could you give a sample? -Wlogical-op merges 2 unrelated warnings:
*) x 2 (you would expect a boolean, not 2, so maybe x2 was meant)
*) x0
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53131
--- Comment #6 from Marc Glisse marc.glisse at normalesup dot org 2012-04-28
12:45:19 UTC ---
(In reply to comment #5)
It seems a pretty small warning, but I guess #1 and #2 could
be split up, if that helps get #2 in.
I think it is the
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53131
Manuel López-Ibáñez manu at gcc dot gnu.org changed:
What|Removed |Added
Keywords||diagnostic
10 matches
Mail list logo
