[Bug c/98029] [11 Regression] volatile triggers incorrect "warning: right-hand operand of comma expression has no effect [-Wunused-value]" since r11-5188-g32934a4f45a72144

2021-01-04 Thread cvs-commit at gcc dot gnu.org via Gcc-bugs 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98029

--- Comment #6 from CVS Commits  ---
The master branch has been updated by Martin Uecker :

https://gcc.gnu.org/g:a000eb5918e09d28838ef572f4eea924d32db09b

commit r11-6450-ga000eb5918e09d28838ef572f4eea924d32db09b
Author: Martin Uecker 
Date:   Mon Jan 4 22:53:58 2021 +0100

C: Add test for incorrect warning for assignment of certain volatile
expressions fixed by commit 58a45ce [PR98029]

2021-01-04  Martin Uecker  

gcc/testsuite/
PR c/98029
* gcc.dg/pr98029.c: New test.

[Bug c/98029] [11 Regression] volatile triggers incorrect "warning: right-hand operand of comma expression has no effect [-Wunused-value]" since r11-5188-g32934a4f45a72144

2020-12-23 Thread marxin at gcc dot gnu.org via Gcc-bugs 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98029

Martin Liška  changed:

   What|Removed |Added

 Resolution|--- |FIXED
 Status|NEW |RESOLVED

--- Comment #5 from Martin Liška  ---
I can confirm that.

[Bug c/98029] [11 Regression] volatile triggers incorrect "warning: right-hand operand of comma expression has no effect [-Wunused-value]" since r11-5188-g32934a4f45a72144

2020-12-23 Thread muecker at gwdg dot de via Gcc-bugs 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98029

--- Comment #4 from Martin Uecker  ---
PATCH https://gcc.gnu.org/pipermail/gcc-patches/2020-December/562465.html

[Bug c/98029] [11 Regression] volatile triggers incorrect "warning: right-hand operand of comma expression has no effect [-Wunused-value]" since r11-5188-g32934a4f45a72144

2020-12-22 Thread muecker at gwdg dot de via Gcc-bugs 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98029

--- Comment #3 from Martin Uecker  ---
It seems this does not happen anymore after fixing PR98047.

[Bug c/98029] [11 Regression] volatile triggers incorrect "warning: right-hand operand of comma expression has no effect [-Wunused-value]" since r11-5188-g32934a4f45a72144

2020-12-07 Thread rguenth at gcc dot gnu.org via Gcc-bugs 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98029

Richard Biener  changed:

   What|Removed |Added

   Target Milestone|--- |11.0

[Bug c/98029] [11 Regression] volatile triggers incorrect "warning: right-hand operand of comma expression has no effect [-Wunused-value]" since r11-5188-g32934a4f45a72144

2020-11-28 Thread uecker at eecs dot berkeley.edu via Gcc-bugs 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98029

Martin Uecker  changed:

   What|Removed |Added

 CC||uecker at eecs dot berkeley.edu

--- Comment #2 from Martin Uecker  ---

It seems with the changes to lvalue conversion the code

volatile int x;
int i;
(x = 1, i = 2);

gets rewritten to

((x = 1, 1), i = 2)

and then build_compound_expr introduces the spurious warning.


There is a another possible related issue:

volatile int x;
(x, 0);

warns about

warning: variable ‘x’ set but not used

[Bug c/98029] [11 Regression] volatile triggers incorrect "warning: right-hand operand of comma expression has no effect [-Wunused-value]" since r11-5188-g32934a4f45a72144

2020-11-27 Thread marxin at gcc dot gnu.org via Gcc-bugs 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98029

Martin Liška  changed:

   What|Removed |Added

 Status|UNCONFIRMED |NEW
 CC||marxin at gcc dot gnu.org
Summary|volatile triggers incorrect |[11 Regression] volatile
   |"warning: right-hand|triggers incorrect
   |operand of comma expression |"warning: right-hand
   |has no effect   |operand of comma expression
   |[-Wunused-value]"   |has no effect
   ||[-Wunused-value]" since
   ||r11-5188-g32934a4f45a72144
 Ever confirmed|0   |1
   Last reconfirmed||2020-11-27

--- Comment #1 from Martin Liška  ---
Confirmed, started with r11-5188-g32934a4f45a72144.