diff options
| author | Richard Kenner <kenner@gcc.gnu.org> | 1997-03-17 13:46:42 -0500 |
|---|---|---|
| committer | Richard Kenner <kenner@gcc.gnu.org> | 1997-03-17 13:46:42 -0500 |
| commit | ae9cb2081c750a64b97c14fea8f49b642ead1343 (patch) | |
| tree | c5b086bf5900c6d02d0eb34abd71f691494bda3d /gcc/c-parse.in | |
| parent | 289d1b21a861305e89c145eed513c4193d7dd63c (diff) | |
| download | gcc-ae9cb2081c750a64b97c14fea8f49b642ead1343.tar.gz | |
(expr_no_commas): Do not store temporary skip_evaluation increments on yacc value stack...
(expr_no_commas): Do not store temporary skip_evaluation increments on
yacc value stack; the code was buggy and occasionally wrote past the
end of the stack.
From-SVN: r13720
Diffstat (limited to 'gcc/c-parse.in')
| -rw-r--r-- | gcc/c-parse.in | 26 |
1 files changed, 11 insertions, 15 deletions
diff --git a/gcc/c-parse.in b/gcc/c-parse.in index 2a40c5a3ce0..e395f86670e 100644 --- a/gcc/c-parse.in +++ b/gcc/c-parse.in @@ -1,5 +1,5 @@ /* YACC parser for C syntax and for Objective C. -*-c-*- - Copyright (C) 1987, 88, 89, 92-5, 1996 Free Software Foundation, Inc. + Copyright (C) 1987, 88, 89, 92-6, 1997 Free Software Foundation, Inc. This file is part of GNU CC. @@ -577,27 +577,24 @@ expr_no_commas: { $$ = parser_build_binary_op ($2, $1, $3); } | expr_no_commas ANDAND { $1 = truthvalue_conversion (default_conversion ($1)); - $<itype>2 = $1 == boolean_false_node; - skip_evaluation += $<itype>2; } + skip_evaluation += $1 == boolean_false_node; } expr_no_commas - { skip_evaluation -= $<itype>2; + { skip_evaluation -= $1 == boolean_false_node; $$ = parser_build_binary_op (TRUTH_ANDIF_EXPR, $1, $4); } | expr_no_commas OROR { $1 = truthvalue_conversion (default_conversion ($1)); - $<itype>3 = $1 == boolean_true_node; - skip_evaluation += $<itype>3; } + skip_evaluation += $1 == boolean_true_node; } expr_no_commas - { skip_evaluation -= $<itype>3; + { skip_evaluation -= $1 == boolean_true_node; $$ = parser_build_binary_op (TRUTH_ORIF_EXPR, $1, $4); } | expr_no_commas '?' { $1 = truthvalue_conversion (default_conversion ($1)); - $<itype>3 = $1 == boolean_true_node; - $<itype>2 = $1 == boolean_false_node; - skip_evaluation += $<itype>2; } + skip_evaluation += $1 == boolean_false_node; } expr ':' - { skip_evaluation += $<itype>3 - $<itype>2; } + { skip_evaluation += (($1 == boolean_true_node) + - ($1 == boolean_false_node)); } expr_no_commas - { skip_evaluation -= $<itype>3; + { skip_evaluation -= $1 == boolean_true_node; $$ = build_conditional_expr ($1, $4, $7); } | expr_no_commas '?' { if (pedantic) @@ -605,10 +602,9 @@ expr_no_commas: /* Make sure first operand is calculated only once. */ $<ttype>2 = save_expr ($1); $1 = truthvalue_conversion (default_conversion ($<ttype>2)); - $<itype>3 = $1 == boolean_true_node; - skip_evaluation += $<itype>3; } + skip_evaluation += $1 == boolean_true_node; } ':' expr_no_commas - { skip_evaluation -= $<itype>3; + { skip_evaluation -= $1 == boolean_true_node; $$ = build_conditional_expr ($1, $<ttype>2, $5); } | expr_no_commas '=' expr_no_commas { $$ = build_modify_expr ($1, NOP_EXPR, $3); |