On Tue, 20 Dec 2016, Jakub Jelinek wrote: > Hi! > > We only record side-effects from the last parameter, the following patch > fixes that by accumulating them from all the parameters. I've checked all > the callers of grokdeclarator and grokdeclarator is always called with expr > either pointing to NULL_TREE, or being NULL, or where we want the > side-effects to be accumulated. Creating a STATEMENT_LIST instead of > COMPOUND_EXPRs doesn't work because we want to mark_exp_read the resulting > expression, which doesn't work for a statement list. COMPOUND_EXPR is what > is used in the two other places where side-effect expressions are > accumulated - one in grokdeclarator, another one in declspecs_add_type. > > Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk?
OK. -- Joseph S. Myers jos...@codesourcery.com
