diff options
| author | Richard Guenther <rguenther@suse.de> | 2008-02-17 14:00:48 +0000 |
|---|---|---|
| committer | Richard Biener <rguenth@gcc.gnu.org> | 2008-02-17 14:00:48 +0000 |
| commit | e09deb1401d411beebe1f56c8435bb9af8e1f4cc (patch) | |
| tree | b460ac23b8786521d83cd102316a7ba02bde87cf /gcc/tree-vrp.c | |
| parent | e6528d32a44a94c5df64cd283428f7121dc66fd7 (diff) | |
| download | gcc-e09deb1401d411beebe1f56c8435bb9af8e1f4cc.zip gcc-e09deb1401d411beebe1f56c8435bb9af8e1f4cc.tar.gz gcc-e09deb1401d411beebe1f56c8435bb9af8e1f4cc.tar.bz2 | |
re PR tree-optimization/35231 (VRP miscompiles libX11)
2008-02-17 Richard Guenther <rguenther@suse.de>
PR tree-optimization/35231
* tree-vrp.c (register_edge_assert_for): Do not assume A == 0
if A | B != 1.
* gcc.c-torture/execute/pr35231.c: New testcase.
From-SVN: r132378
Diffstat (limited to 'gcc/tree-vrp.c')
| -rw-r--r-- | gcc/tree-vrp.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/gcc/tree-vrp.c b/gcc/tree-vrp.c index 5f9a327..f9615d1 100644 --- a/gcc/tree-vrp.c +++ b/gcc/tree-vrp.c @@ -3753,7 +3753,11 @@ register_edge_assert_for (tree name, edge e, block_stmt_iterator si, tree cond) if (TREE_CODE (def_stmt) == GIMPLE_MODIFY_STMT && (TREE_CODE (GIMPLE_STMT_OPERAND (def_stmt, 1)) == TRUTH_OR_EXPR - || TREE_CODE (GIMPLE_STMT_OPERAND (def_stmt, 1)) == BIT_IOR_EXPR)) + /* For BIT_IOR_EXPR only if NAME == 0 both operands have + necessarily zero value. */ + || (comp_code == EQ_EXPR + && (TREE_CODE (GIMPLE_STMT_OPERAND (def_stmt, 1)) + == BIT_IOR_EXPR)))) { tree op0 = TREE_OPERAND (GIMPLE_STMT_OPERAND (def_stmt, 1), 0); tree op1 = TREE_OPERAND (GIMPLE_STMT_OPERAND (def_stmt, 1), 1); |