diff options
Diffstat (limited to 'gcc')
| -rw-r--r-- | gcc/ChangeLog | 6 | ||||
| -rw-r--r-- | gcc/c-typeck.c | 8 | ||||
| -rw-r--r-- | gcc/testsuite/ChangeLog | 5 | ||||
| -rw-r--r-- | gcc/testsuite/gcc.c-torture/compile/pr45969-1.c | 6 |
4 files changed, 24 insertions, 1 deletions
diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 6c92f30..303ea78 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,9 @@ +2010-10-14 Joseph Myers <joseph@codesourcery.com> + + PR c/45969 + * c-typeck.c (build_binary_op): Don't try to compute a semantic + type with excess precision for boolean operations. + 2010-10-14 Jeremie Salvucci <jeremie.salvucci@free.fr> Basile Starynkevitch <basile@starynkevitch.net> diff --git a/gcc/c-typeck.c b/gcc/c-typeck.c index 14bc281..fe10e8b 100644 --- a/gcc/c-typeck.c +++ b/gcc/c-typeck.c @@ -9423,6 +9423,10 @@ build_binary_op (location_t location, enum tree_code code, precision. */ bool may_need_excess_precision; + /* True means this is a boolean operation that converts both its + operands to truth-values. */ + bool boolean_op = false; + if (location == UNKNOWN_LOCATION) location = input_location; @@ -9650,6 +9654,7 @@ build_binary_op (location_t location, enum tree_code code, op0 = c_common_truthvalue_conversion (location, op0); op1 = c_common_truthvalue_conversion (location, op1); converted = 1; + boolean_op = true; } if (code == TRUTH_ANDIF_EXPR) { @@ -10192,7 +10197,8 @@ build_binary_op (location_t location, enum tree_code code, if (build_type == NULL_TREE) { build_type = result_type; - if (type0 != orig_type0 || type1 != orig_type1) + if ((type0 != orig_type0 || type1 != orig_type1) + && !boolean_op) { gcc_assert (may_need_excess_precision && common); semantic_result_type = c_common_type (orig_type0, orig_type1); diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index fe240d4..a81938d 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,8 @@ +2010-10-14 Joseph Myers <joseph@codesourcery.com> + + PR c/45969 + * gcc.c-torture/compile/pr45969-1.c: New test. + 2010-10-14 Richard Guenther <rguenther@suse.de> PR lto/45382 diff --git a/gcc/testsuite/gcc.c-torture/compile/pr45969-1.c b/gcc/testsuite/gcc.c-torture/compile/pr45969-1.c new file mode 100644 index 0000000..ccf68bc --- /dev/null +++ b/gcc/testsuite/gcc.c-torture/compile/pr45969-1.c @@ -0,0 +1,6 @@ +/* { dg-options "-std=c89" } */ +void crash() { + double l[4]; + if((l[0]+l[2]) && (l[1]+l[3])){ + } +} |