[Bug c/59871] [4.7/4.8/4.9 Regression] No unused value warning for comma expression

2014-01-23 Thread mpolacek at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59871 --- Comment #7 from Marek Polacek mpolacek at gcc dot gnu.org --- Author: mpolacek Date: Thu Jan 23 19:04:29 2014 New Revision: 207002 URL: http://gcc.gnu.org/viewcvs?rev=207002root=gccview=rev Log: PR c/59871 c/ * c-typeck.c

[Bug c/59871] [4.7/4.8/4.9 Regression] No unused value warning for comma expression

2014-01-23 Thread mpolacek at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59871 Marek Polacek mpolacek at gcc dot gnu.org changed: What|Removed |Added Status|ASSIGNED|RESOLVED

[Bug c/59871] [4.7/4.8/4.9 Regression] No unused value warning for comma expression

2014-01-18 Thread jakub at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59871 Jakub Jelinek jakub at gcc dot gnu.org changed: What|Removed |Added Summary|No unused value warning for |[4.7/4.8/4.9

[Bug c/59871] [4.7/4.8/4.9 Regression] No unused value warning for comma expression

2014-01-18 Thread jakub at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59871 --- Comment #4 from Jakub Jelinek jakub at gcc dot gnu.org --- I'd say easiest would be at the spot where we warn currently about the left-hand ... check if the left hand operand is a COMPOUND_EXPR, in that case find the rightmost operand of all

[Bug c/59871] [4.7/4.8/4.9 Regression] No unused value warning for comma expression

2014-01-18 Thread jakub at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59871 --- Comment #5 from Jakub Jelinek jakub at gcc dot gnu.org --- And similarly in emit_side_effect_warnings, if there are TREE_SIDE_EFFECTS, but the expr is COMPOUND_EXPR, find the rightmost operand of the nested COMPOUND_EXPRs and if it doesn't

[Bug c/59871] [4.7/4.8/4.9 Regression] No unused value warning for comma expression

2014-01-18 Thread mpolacek at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59871 Marek Polacek mpolacek at gcc dot gnu.org changed: What|Removed |Added Status|NEW |ASSIGNED