OpenE2K
/
gcc
2
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

* tree.c (save_expr): Don't fold a COMPONENT_REF. 

From-SVN: r66264
This commit is contained in:
Richard Kenner 2003-04-29 22:38:09 +00:00 committed by Richard Kenner
parent 1f6f0cb610
commit 84d8756de1
2 changed files with 13 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
gcc/ChangeLog
View File

@ -1,3 +1,7 @@
2003-04-29 Richard Kenner <kenner@vlsi1.ultra.nyu.edu>
* tree.c (save_expr): Don't fold a COMPONENT_REF.
2003-04-29 Olivier Hainque <hainque@act-europe.fr>
* calls.c (expand_call): When modes of target and valreg match, force

11
gcc/tree.c
View File

@ -1378,14 +1378,21 @@ tree
save_expr (expr)
tree expr;
{
tree t = fold (expr);
tree inner = skip_simple_arithmetic (t);
tree t = expr;
tree inner;
/* Don't fold a COMPONENT_EXPR: if the operand was a CONSTRUCTOR (the
only time it will fold), it can cause problems with PLACEHOLDER_EXPRs
in Ada. Moreover, it isn't at all clear why we fold here at all. */
if (TREE_CODE (t) != COMPONENT_REF)
t = fold (t);
/* If the tree evaluates to a constant, then we don't want to hide that
fact (i.e. this allows further folding, and direct checks for constants).
However, a read-only object that has side effects cannot be bypassed.
Since it is no problem to reevaluate literals, we just return the
literal node. */
inner = skip_simple_arithmetic (t);
if (TREE_CONSTANT (inner)
|| (TREE_READONLY (inner) && ! TREE_SIDE_EFFECTS (inner))
|| TREE_CODE (inner) == SAVE_EXPR