Commit caaf2272 by Jakub Jelinek Committed by Jakub Jelinek

re PR c/5420 (bad code generated with gcc3.1/ia64)

	PR c/5420:
	* c-common.c (c_unsafe_for_reeval): Make COMPOUND_LITERAL_EXPR
	unsafe for reevaluation.

	* gcc.c-torture/execute/20020206-2.c: New test.

From-SVN: r49550
parent d5129288
2002-02-06 Jakub Jelinek <jakub@redhat.com> 2002-02-06 Jakub Jelinek <jakub@redhat.com>
PR c/5420:
* c-common.c (c_unsafe_for_reeval): Make COMPOUND_LITERAL_EXPR
unsafe for reevaluation.
2002-02-06 Jakub Jelinek <jakub@redhat.com>
PR c/5482: PR c/5482:
* c-common.c (c_expand_expr) [STMT_EXPR]: If last expression is not * c-common.c (c_expand_expr) [STMT_EXPR]: If last expression is not
EXPR_STMT, but COMPOUND_STMT, recurse into it. EXPR_STMT, but COMPOUND_STMT, recurse into it.
......
...@@ -3567,8 +3567,10 @@ int ...@@ -3567,8 +3567,10 @@ int
c_unsafe_for_reeval (exp) c_unsafe_for_reeval (exp)
tree exp; tree exp;
{ {
/* Statement expressions may not be reevaluated. */ /* Statement expressions may not be reevaluated, likewise compound
if (TREE_CODE (exp) == STMT_EXPR) literals. */
if (TREE_CODE (exp) == STMT_EXPR
|| TREE_CODE (exp) == COMPOUND_LITERAL_EXPR)
return 2; return 2;
/* Walk all other expressions. */ /* Walk all other expressions. */
......
...@@ -2,6 +2,8 @@ ...@@ -2,6 +2,8 @@
* gcc.c-torture/execute/20020206-1.c: New test. * gcc.c-torture/execute/20020206-1.c: New test.
* gcc.c-torture/execute/20020206-2.c: New test.
PR optimization/5429: PR optimization/5429:
* gcc.c-torture/compile/20020206-1.c: New test. * gcc.c-torture/compile/20020206-1.c: New test.
......
/* Origin: PR c/5420 from David Mosberger <davidm@hpl.hp.com>.
This testcase was miscompiled when tail call optimizing, because a
compound literal initialization was emitted only in the tail call insn
chain, not in the normal call insn chain. */
typedef struct { unsigned short a; } A;
extern void abort (void);
extern void exit (int);
void foo (unsigned int x)
{
if (x != 0x800 && x != 0x810)
abort ();
}
int
main (int argc, char **argv)
{
int i;
for (i = 0; i < 2; ++i)
foo (((A) { ((!(i >> 4) ? 8 : 64 + (i >> 4)) << 8) + (i << 4) } ).a);
exit (0);
}
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment