diff options
author | Jakub Jelinek <jakub@redhat.com> | 2013-09-09 19:09:59 +0200 |
---|---|---|
committer | Jakub Jelinek <jakub@gcc.gnu.org> | 2013-09-09 19:09:59 +0200 |
commit | 28fd0ba2d17a09ddf5106f1e60c475a93fba3090 (patch) | |
tree | 0a438d585a5e277f8a5489576bb8698c82b76786 /gcc/tree-ssa-reassoc.c | |
parent | 552d99fe881c262ec96235e9e8f00fc54809f458 (diff) | |
download | gcc-28fd0ba2d17a09ddf5106f1e60c475a93fba3090.zip gcc-28fd0ba2d17a09ddf5106f1e60c475a93fba3090.tar.gz gcc-28fd0ba2d17a09ddf5106f1e60c475a93fba3090.tar.bz2 |
re PR tree-optimization/58364 (likely wrong code bug)
PR tree-optimization/58364
* tree-ssa-reassoc.c (init_range_entry): For BIT_NOT_EXPR on
BOOLEAN_TYPE, only invert in_p and continue with arg0 if
the current range can't be an unconditional true or false.
* gcc.c-torture/execute/pr58364.c: New test.
From-SVN: r202409
Diffstat (limited to 'gcc/tree-ssa-reassoc.c')
-rw-r--r-- | gcc/tree-ssa-reassoc.c | 9 |
1 files changed, 8 insertions, 1 deletions
diff --git a/gcc/tree-ssa-reassoc.c b/gcc/tree-ssa-reassoc.c index 03c6c5d..b4a3b0a 100644 --- a/gcc/tree-ssa-reassoc.c +++ b/gcc/tree-ssa-reassoc.c @@ -1801,7 +1801,14 @@ init_range_entry (struct range_entry *r, tree exp, gimple stmt) switch (code) { case BIT_NOT_EXPR: - if (TREE_CODE (TREE_TYPE (exp)) == BOOLEAN_TYPE) + if (TREE_CODE (TREE_TYPE (exp)) == BOOLEAN_TYPE + /* Ensure the range is either +[-,0], +[0,0], + -[-,0], -[0,0] or +[1,-], +[1,1], -[1,-] or + -[1,1]. If it is e.g. +[-,-] or -[-,-] + or similar expression of unconditional true or + false, it should not be negated. */ + && ((high && integer_zerop (high)) + || (low && integer_onep (low)))) { in_p = !in_p; exp = arg0; |