diff options
| author | Richard Biener <rguenther@suse.de> | 2015-07-16 11:27:34 +0000 |
|---|---|---|
| committer | Richard Biener <rguenth@gcc.gnu.org> | 2015-07-16 11:27:34 +0000 |
| commit | ef3b59ace9400afc4beba51f7ca91a1dfcc3ca38 (patch) | |
| tree | a442c599fd189235da985453491c9d264689e929 | |
| parent | fb0b291494898e922f9b8def65da1702d2ae756e (diff) | |
| download | gcc-ef3b59ace9400afc4beba51f7ca91a1dfcc3ca38.zip gcc-ef3b59ace9400afc4beba51f7ca91a1dfcc3ca38.tar.gz gcc-ef3b59ace9400afc4beba51f7ca91a1dfcc3ca38.tar.bz2 | |
re PR tree-optimization/66894 (wrong code at -Os and above on x86_64-linux-gnu)
2015-07-16 Richard Biener <rguenther@suse.de>
PR tree-optimization/66894
* tree-vrp.c (register_edge_assert_for_2): Fix bad assumption
about deriving NE_EXPR from truncated values.
* gcc.dg/torture/pr66894.c: New testcase.
From-SVN: r225870
| -rw-r--r-- | gcc/ChangeLog | 6 | ||||
| -rw-r--r-- | gcc/testsuite/ChangeLog | 5 | ||||
| -rw-r--r-- | gcc/testsuite/gcc.dg/torture/pr66894.c | 21 | ||||
| -rw-r--r-- | gcc/tree-vrp.c | 6 |
4 files changed, 34 insertions, 4 deletions
diff --git a/gcc/ChangeLog b/gcc/ChangeLog index e945a28..1d5434c 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,9 @@ +2015-07-16 Richard Biener <rguenther@suse.de> + + PR tree-optimization/66894 + * tree-vrp.c (register_edge_assert_for_2): Fix bad assumption + about deriving NE_EXPR from truncated values. + 2015-07-16 Martin Liska <mliska@suse.cz> * alloc-pool.h diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index 8bc9341..9bace80 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,5 +1,10 @@ 2015-07-16 Richard Biener <rguenther@suse.de> + PR tree-optimization/66894 + * gcc.dg/torture/pr66894.c: New testcase. + +2015-07-16 Richard Biener <rguenther@suse.de> + * gcc.dg/tree-ssa/pr21031.c: Adjust. 2015-07-16 Uros Bizjak <ubizjak@gmail.com> diff --git a/gcc/testsuite/gcc.dg/torture/pr66894.c b/gcc/testsuite/gcc.dg/torture/pr66894.c new file mode 100644 index 0000000..b544940 --- /dev/null +++ b/gcc/testsuite/gcc.dg/torture/pr66894.c @@ -0,0 +1,21 @@ +/* { dg-do run } */ + +short a, b; + +int +main () +{ + for (; a != 1; a += 3) + { + int c = 0; + for (; c < 2; c++) + if (a) + { + char d = a; + b = d ? 1 / d : 0; + } + else + break; + } + return 0; +} diff --git a/gcc/tree-vrp.c b/gcc/tree-vrp.c index 795b027..d962683 100644 --- a/gcc/tree-vrp.c +++ b/gcc/tree-vrp.c @@ -5382,13 +5382,11 @@ register_edge_assert_for_2 (tree name, edge e, gimple_stmt_iterator bsi, } else if (CONVERT_EXPR_CODE_P (code)) { - /* For truncating conversions require that the constant - fits in the truncated type if we are going to record + /* For truncating conversions we cannot record an inequality. */ if (comp_code == NE_EXPR && (TYPE_PRECISION (TREE_TYPE (name2)) - < TYPE_PRECISION (TREE_TYPE (name))) - && ! int_fits_type_p (val, TREE_TYPE (name2))) + < TYPE_PRECISION (TREE_TYPE (name)))) continue; cst = fold_convert (TREE_TYPE (name2), val); } |