[Bug c++/70336] [5/6 regression] Incorrect Wconversion warning

2016-04-05 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70336 --- Comment #11 from Jakub Jelinek --- Author: jakub Date: Tue Apr 5 21:33:37 2016 New Revision: 234764 URL: https://gcc.gnu.org/viewcvs?rev=234764&root=gcc&view=rev Log: PR c++/70336 * match.pd (nested int casts): Limit to GIMP

[Bug c++/70336] [5/6 regression] Incorrect Wconversion warning

2016-04-04 Thread manu at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70336 --- Comment #10 from Manuel López-Ibáñez --- (In reply to Jakub Jelinek from comment #8) > Here -Wconversion will warn for many casesm even in 4.9, eventhough one > could argue that say in the f4 case nothing is lost during conversion, or There

[Bug c++/70336] [5/6 regression] Incorrect Wconversion warning

2016-04-04 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70336 --- Comment #9 from Jakub Jelinek --- (In reply to Manuel López-Ibáñez from comment #7) > (In reply to Jakub Jelinek from comment #6) > > Created attachment 38173 [details] > > gcc6-pr70336.patch > > > > Patch to revert (for GENERIC only) the ma

[Bug c++/70336] [5/6 regression] Incorrect Wconversion warning

2016-04-04 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70336 --- Comment #8 from Jakub Jelinek --- The above patch (untested so far) should get rid of the regression, thus we'd warn exactly where 4.9 warned. But, consider say: void f1 (unsigned char * x, int y, int z) { x[z / 8] |= (unsigned char) (0x80

[Bug c++/70336] [5/6 regression] Incorrect Wconversion warning

2016-04-04 Thread manu at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70336 --- Comment #7 from Manuel López-Ibáñez --- (In reply to Jakub Jelinek from comment #6) > Created attachment 38173 [details] > gcc6-pr70336.patch > > Patch to revert (for GENERIC only) the match.pd change that causes this > regression. Does it

[Bug c++/70336] [5/6 regression] Incorrect Wconversion warning

2016-04-04 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70336 --- Comment #6 from Jakub Jelinek --- Created attachment 38173 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=38173&action=edit gcc6-pr70336.patch Patch to revert (for GENERIC only) the match.pd change that causes this regression.

[Bug c++/70336] [5/6 regression] Incorrect Wconversion warning

2016-04-04 Thread manu at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70336 --- Comment #5 from Manuel López-Ibáñez --- (In reply to Jakub Jelinek from comment #4) > Created attachment 38172 [details] > gcc6-pr70336.patch > > IMNSHO -Wconversion is totally useless warning, and one where if you want to > avoid "false pos

[Bug c++/70336] [5/6 regression] Incorrect Wconversion warning

2016-04-04 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70336 --- Comment #4 from Jakub Jelinek --- Created attachment 38172 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=38172&action=edit gcc6-pr70336.patch IMNSHO -Wconversion is totally useless warning, and one where if you want to avoid "false po

[Bug c++/70336] [5/6 regression] Incorrect Wconversion warning

2016-04-04 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70336 Jakub Jelinek changed: What|Removed |Added CC||jakub at gcc dot gnu.org --- Comment #3

[Bug c++/70336] [5/6 regression] Incorrect Wconversion warning

2016-04-03 Thread ppalka at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70336 Patrick Palka changed: What|Removed |Added CC||ppalka at gcc dot gnu.org --- Comment #2

[Bug c++/70336] [5/6 regression] Incorrect Wconversion warning

2016-04-01 Thread manu at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70336 Manuel López-Ibáñez changed: What|Removed |Added Keywords||diagnostic Status|UNCO