Skip to content

Commit

Permalink
2015-07-14 Richard Biener <[email protected]>
Browse files Browse the repository at this point in the history
	PR tree-optimization/66863
	* tree-vrp.c (register_edge_assert_for_2): Properly restrict
	what we record for conversion use stmt lhs inequalities.

	* gcc.dg/torture/pr66863.c: New testcase.


git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@225768 138bc75d-0d04-0410-961f-82ee72b054a4
  • Loading branch information
rguenth committed Jul 14, 2015
1 parent 6700b27 commit 6c2daf3
Show file tree
Hide file tree
Showing 4 changed files with 47 additions and 1 deletion.
6 changes: 6 additions & 0 deletions gcc/ChangeLog
Original file line number Diff line number Diff line change
@@ -1,3 +1,9 @@
2015-07-14 Richard Biener <[email protected]>

PR tree-optimization/66863
* tree-vrp.c (register_edge_assert_for_2): Properly restrict
what we record for conversion use stmt lhs inequalities.

2015-07-14 Richard Biener <[email protected]>

* genmatch.c (dt_node::gen_kids_1): Fix case label indenting.
Expand Down
5 changes: 5 additions & 0 deletions gcc/testsuite/ChangeLog
Original file line number Diff line number Diff line change
@@ -1,3 +1,8 @@
2015-07-14 Richard Biener <[email protected]>

PR tree-optimization/66863
* gcc.dg/torture/pr66863.c: New testcase.

2015-07-13 Patrick Palka <[email protected]>

PR c++/65186
Expand Down
25 changes: 25 additions & 0 deletions gcc/testsuite/gcc.dg/torture/pr66863.c
Original file line number Diff line number Diff line change
@@ -0,0 +1,25 @@
/* { dg-do run } */

int a, b;

int
fn1 (int p1)
{
if (p1 < -2147483647)
return 0;
else
return 1;
}

int
fn2 (int p1, short p2)
{
return p2 ? p1 % p2 : 0;
}

int
main ()
{
b = fn2 (fn1 (a), a);
return 0;
}
12 changes: 11 additions & 1 deletion gcc/tree-vrp.c
Original file line number Diff line number Diff line change
Expand Up @@ -5381,7 +5381,17 @@ register_edge_assert_for_2 (tree name, edge e, gimple_stmt_iterator bsi,
cst = int_const_binop (code, val, cst);
}
else if (CONVERT_EXPR_CODE_P (code))
cst = fold_convert (TREE_TYPE (name2), val);
{
/* For truncating conversions require that the constant
fits in the truncated type if we are going to 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)))
continue;
cst = fold_convert (TREE_TYPE (name2), val);
}
else
continue;

Expand Down

0 comments on commit 6c2daf3

Please sign in to comment.