diff options
| author | David Pagan <dave.pagan@oracle.com> | 2018-03-13 18:10:09 +0000 |
|---|---|---|
| committer | Joseph Myers <jsm28@gcc.gnu.org> | 2018-03-13 18:10:09 +0000 |
| commit | ada6bad9780d405aea40acbea4f0813b05e3733a (patch) | |
| tree | fb467b2bf55a3765f69f31dbf4f572aef4e42353 /gcc/fortran/iresolve.c | |
| parent | b34f5c5e69bb024f41b4e15869f7ff18c3f8dd6f (diff) | |
| download | gcc-ada6bad9780d405aea40acbea4f0813b05e3733a.zip gcc-ada6bad9780d405aea40acbea4f0813b05e3733a.tar.gz gcc-ada6bad9780d405aea40acbea4f0813b05e3733a.tar.bz2 | |
PR c/46921 Lost side effect when struct initializer expression uses comma operator
This patch fixes improper handling of comma operator expression in a
struct field initializer as described in:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=46921
Currently, function output_init_element () does not evaluate the left
hand expression in a comma operator that's used for a struct
initializer field if the right hand side is zero-sized. However, the
left hand expression must be evaluated if it's found to have side
effects (for example, a function call).
Patch was successfully bootstrapped and tested on x86_64-linux.
gcc/c:
2018-03-13 David Pagan <dave.pagan@oracle.com>
PR c/46921
* c-typeck.c (output_init_element): Ensure field initializer
expression is always evaluated if there are side effects.
gcc/testsuite:
2018-03-13 David Pagan <dave.pagan@oracle.com>
PR c/46921
* gcc.dg/pr46921.c: New test.
From-SVN: r258497
Diffstat (limited to 'gcc/fortran/iresolve.c')
0 files changed, 0 insertions, 0 deletions