Summary: | [4.9/4.10 Regression] wrong code with -O2 -fno-early-inlining (maybe wrong devirtualization) | ||
---|---|---|---|
Product: | gcc | Reporter: | Zdenek Sojka <zsojka> |
Component: | ipa | Assignee: | Martin Jambor <jamborm> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | hubicka, jakub, jamborm |
Priority: | P2 | Keywords: | wrong-code |
Version: | 5.0 | ||
Target Milestone: | 4.9.1 | ||
Host: | x86_64-pc-linux-gnu | Target: | x86_64-pc-linux-gnu |
Build: | Known to work: | 4.7.4, 4.8.3 | |
Known to fail: | 4.10.0, 4.9.1 | Last reconfirmed: | 2014-05-07 00:00:00 |
Attachments: | reduced testcase |
Description
Zdenek Sojka
2014-05-06 19:48:13 UTC
Confirmed. Mine I have proposed a fix on the mailing list: https://gcc.gnu.org/ml/gcc-patches/2014-05/msg01199.html Author: jamborm Date: Thu May 15 15:04:18 2014 New Revision: 210477 URL: http://gcc.gnu.org/viewcvs?rev=210477&root=gcc&view=rev Log: 2014-05-15 Martin Jambor <mjambor@suse.cz> PR ipa/61085 * ipa-prop.c (update_indirect_edges_after_inlining): Check type_preserved flag when the indirect edge is polymorphic. testsuite/ * g++.dg/ipa/pr61085.C: New test. Added: trunk/gcc/testsuite/g++.dg/ipa/pr61085.C Modified: trunk/gcc/ChangeLog trunk/gcc/ipa-prop.c trunk/gcc/testsuite/ChangeLog Author: jamborm Date: Thu May 15 15:10:58 2014 New Revision: 210478 URL: http://gcc.gnu.org/viewcvs?rev=210478&root=gcc&view=rev Log: 2014-05-15 Martin Jambor <mjambor@suse.cz> PR ipa/61085 * ipa-prop.c (update_indirect_edges_after_inlining): Check type_preserved flag when the indirect edge is polymorphic. testsuite/ * g++.dg/ipa/pr61085.C: New test. Added: branches/gcc-4_9-branch/gcc/testsuite/g++.dg/ipa/pr61085.C Modified: branches/gcc-4_9-branch/gcc/ChangeLog branches/gcc-4_9-branch/gcc/ipa-prop.c branches/gcc-4_9-branch/gcc/testsuite/ChangeLog Fixed. Can you open a new bug for that? (In reply to Richard Biener from comment #9) > Can you open a new bug for that? Opened PR61884 for that. |