diff options
| -rw-r--r-- | gcc/ChangeLog | 6 | ||||
| -rw-r--r-- | gcc/ipa-split.c | 3 | ||||
| -rw-r--r-- | gcc/testsuite/ChangeLog | 5 | ||||
| -rw-r--r-- | gcc/testsuite/g++.dg/ipa/pr80212.C | 18 |
4 files changed, 32 insertions, 0 deletions
diff --git a/gcc/ChangeLog b/gcc/ChangeLog index e6ad7ce..6237a2c 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,9 @@ +2017-04-07 Martin Liska <mliska@suse.cz> + + PR ipa/80212 + * ipa-split.c (split_function): Add function part to a same comdat + group. + 2017-04-07 Richard Biener <rguenther@suse.de> PR middle-end/80341 diff --git a/gcc/ipa-split.c b/gcc/ipa-split.c index da3c2c6..ae1de6f 100644 --- a/gcc/ipa-split.c +++ b/gcc/ipa-split.c @@ -1363,6 +1363,9 @@ split_function (basic_block return_bb, struct split_point *split_point, /* Let's take a time profile for splitted function. */ node->tp_first_run = cur_node->tp_first_run + 1; + if (cur_node->same_comdat_group) + node->add_to_same_comdat_group (cur_node); + /* For usual cloning it is enough to clear builtin only when signature changes. For partial inlining we however can not expect the part of builtin implementation to have same semantic as the whole. */ diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index 565485a..43e7bec 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,8 @@ +2017-04-07 Martin Liska <mliska@suse.cz> + + PR ipa/80212 + * g++.dg/ipa/pr80212.C: New test. + 2017-04-07 Thomas Preud'homme <thomas.preudhomme@arm.com> * gcc.dg/tree-ssa/pr79800.c: Require c99_runtime. diff --git a/gcc/testsuite/g++.dg/ipa/pr80212.C b/gcc/testsuite/g++.dg/ipa/pr80212.C new file mode 100644 index 0000000..60d3b61 --- /dev/null +++ b/gcc/testsuite/g++.dg/ipa/pr80212.C @@ -0,0 +1,18 @@ +// PR ipa/80212 +// { dg-options "-O2 --param partial-inlining-entry-probability=403796683 -fno-early-inlining" } + +struct b +{ + virtual b *c () const; +}; +struct d : virtual b +{ +}; +struct e : d +{ + e * + c () const + { + } +}; +main () { e a; } |