diff options
author | Richard Guenther <rguenther@suse.de> | 2011-09-05 09:03:08 +0000 |
---|---|---|
committer | Richard Biener <rguenth@gcc.gnu.org> | 2011-09-05 09:03:08 +0000 |
commit | 830c740f17a21a901b91de6723d97567bae47770 (patch) | |
tree | ea45e4b0b22441a465d4935a39d510d54d5b860a /gcc | |
parent | c149e48765862535842cf968dc10667038988606 (diff) | |
download | gcc-830c740f17a21a901b91de6723d97567bae47770.zip gcc-830c740f17a21a901b91de6723d97567bae47770.tar.gz gcc-830c740f17a21a901b91de6723d97567bae47770.tar.bz2 |
stor-layout.c (layout_type): Use size_binop for array size calculations.
2011-09-05 Richard Guenther <rguenther@suse.de>
* stor-layout.c (layout_type): Use size_binop for array size
calculations.
c-family/
* c-common.c (complete_array_type): Use ssize_int (-1) instead
of integer_minus_one_node for empty array upper bounds.
From-SVN: r178526
Diffstat (limited to 'gcc')
-rw-r--r-- | gcc/ChangeLog | 5 | ||||
-rw-r--r-- | gcc/c-family/ChangeLog | 5 | ||||
-rw-r--r-- | gcc/c-family/c-common.c | 2 | ||||
-rw-r--r-- | gcc/stor-layout.c | 13 |
4 files changed, 17 insertions, 8 deletions
diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 5a9a339..ab05faa 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,8 @@ +2011-09-05 Richard Guenther <rguenther@suse.de> + + * stor-layout.c (layout_type): Use size_binop for array size + calculations. + 2011-09-05 Georg-Johann Lay <avr@gjlay.de> * config/avr/avr.h (progmem_section): Remove Declaration. diff --git a/gcc/c-family/ChangeLog b/gcc/c-family/ChangeLog index 44eae08..95abde6 100644 --- a/gcc/c-family/ChangeLog +++ b/gcc/c-family/ChangeLog @@ -1,3 +1,8 @@ +2011-09-05 Richard Guenther <rguenther@suse.de> + + * c-common.c (complete_array_type): Use ssize_int (-1) instead + of integer_minus_one_node for empty array upper bounds. + 2011-08-28 Dodji Seketeli <dodji@redhat.com> * c-pch.c (c_common_read_pch): Call linemap_add with LC_ENTER as diff --git a/gcc/c-family/c-common.c b/gcc/c-family/c-common.c index 4cace8d..9c42d594 100644 --- a/gcc/c-family/c-common.c +++ b/gcc/c-family/c-common.c @@ -8844,7 +8844,7 @@ complete_array_type (tree *ptype, tree initial_value, bool do_default) { if (pedantic) failure = 3; - maxindex = integer_minus_one_node; + maxindex = ssize_int (-1); } else { diff --git a/gcc/stor-layout.c b/gcc/stor-layout.c index f15da0c..359541e 100644 --- a/gcc/stor-layout.c +++ b/gcc/stor-layout.c @@ -1959,16 +1959,15 @@ layout_type (tree type) if (integer_zerop (element_size)) length = size_zero_node; - /* The computation should happen in the original type so - that (possible) negative values are handled appropriately. */ + /* The computation should happen in the original signedness so + that (possible) negative values are handled appropriately + when determining overflow. */ else length = fold_convert (sizetype, - fold_build2 (PLUS_EXPR, TREE_TYPE (lb), - build_int_cst (TREE_TYPE (lb), 1), - fold_build2 (MINUS_EXPR, - TREE_TYPE (lb), - ub, lb))); + size_binop (PLUS_EXPR, + build_int_cst (TREE_TYPE (lb), 1), + size_binop (MINUS_EXPR, ub, lb))); TYPE_SIZE (type) = size_binop (MULT_EXPR, element_size, fold_convert (bitsizetype, |