From afe1a916d7e6195a85569d33685fd3259f9a0e84 Mon Sep 17 00:00:00 2001 From: Richard Guenther Date: Fri, 5 Jan 2007 21:04:23 +0000 Subject: [PATCH] re PR c++/27826 (ICE in copy_to_mode_reg) 2007-01-05 Richard Guenther PR middle-end/27826 * tree.c (get_narrower): Do not construct COMPONENT_REFs with mismatched types. Instead explicitly build a conversion NOP_EXPR. * g++.dg/opt/pr27826.C: New testcase. From-SVN: r120498 --- gcc/ChangeLog | 7 +++++++ gcc/testsuite/ChangeLog | 5 +++++ gcc/testsuite/g++.dg/opt/pr27826.C | 14 ++++++++++++++ gcc/tree.c | 6 ++---- 4 files changed, 28 insertions(+), 4 deletions(-) create mode 100644 gcc/testsuite/g++.dg/opt/pr27826.C diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 92925123a4a..45de0c4bca4 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,10 @@ +2007-01-05 Richard Guenther + + PR middle-end/27826 + * tree.c (get_narrower): Do not construct COMPONENT_REFs + with mismatched types. Instead explicitly build a + conversion NOP_EXPR. + 2007-01-05 Ian Lance Taylor * c-common.c (decl_with_nonnull_addr_p): New function. diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index 8856b7e2ff6..817846a9fb8 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,8 @@ +2007-01-05 Richard Guenther + + PR middle-end/27826 + * g++.dg/opt/pr27826.C: New testcase. + 2007-01-05 Ian Lance Taylor * gcc.dg/Walways-true-1.c: New test. diff --git a/gcc/testsuite/g++.dg/opt/pr27826.C b/gcc/testsuite/g++.dg/opt/pr27826.C new file mode 100644 index 00000000000..5e40f1746f9 --- /dev/null +++ b/gcc/testsuite/g++.dg/opt/pr27826.C @@ -0,0 +1,14 @@ +/* { dg-do compile } */ +/* { dg-options "-O3" } */ + +struct Geometry +{ + int type:16; +}; +struct Geometry get() {}; +int f() +{ + struct Geometry test; + return get().type == test.type; +} + diff --git a/gcc/tree.c b/gcc/tree.c index 5abd6f345f3..71d221ee486 100644 --- a/gcc/tree.c +++ b/gcc/tree.c @@ -5989,12 +5989,10 @@ get_narrower (tree op, int *unsignedp_ptr) { if (first) uns = DECL_UNSIGNED (TREE_OPERAND (op, 1)); - win = build3 (COMPONENT_REF, type, TREE_OPERAND (op, 0), - TREE_OPERAND (op, 1), NULL_TREE); - TREE_SIDE_EFFECTS (win) = TREE_SIDE_EFFECTS (op); - TREE_THIS_VOLATILE (win) = TREE_THIS_VOLATILE (op); + win = fold_convert (type, op); } } + *unsignedp_ptr = uns; return win; } -- 2.30.2