https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84044
Jan Hubicka changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84044
--- Comment #14 from Jan Hubicka ---
Author: hubicka
Date: Wed Nov 21 02:38:43 2018
New Revision: 266334
URL: https://gcc.gnu.org/viewcvs?rev=266334=gcc=rev
Log:
PR lto/84044
* ipa-devirt.c (odr_types_equivalent_p): Use
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84044
--- Comment #13 from Jan Hubicka ---
... and I can also confirm that the original testcase no longer triggers.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84044
--- Comment #12 from Jan Hubicka ---
Actually the issue here is that enum is context that is the type B and it has
keyed vtable which makes it unmerged. So we are correct to not merge here
provided that we want to keep TYPE_CONTEXT here (which
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84044
--- Comment #11 from Jan Hubicka ---
Warning from #9 is caused by fact that I compare pointers rather than test for
equality so when the enums are not merged, they triggers the warning even if
the values are in fact equivalent.
The following
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84044
--- Comment #10 from Martin Liška ---
> Thread model: posix
> gcc version 9.0.0 20181110 (experimental) [trunk revision 266003] (Debian
> 20181110-2)
>
> Regards,
> Arnaud Giersch
The warning appeared again on trunk in r265875
Author: hubicka
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84044
Arnaud Giersch changed:
What|Removed |Added
CC||arnaud.giersch at free dot fr
---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84044
Martin Liška changed:
What|Removed |Added
Status|NEW |ASSIGNED
Assignee|unassigned
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84044
--- Comment #8 from Jan Hubicka ---
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84044
>
> --- Comment #7 from rguenther at suse dot de ---
> On Fri, 26 Jan 2018, marxin at gcc dot gnu.org wrote:
>
> >
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84044
--- Comment #7 from rguenther at suse dot de ---
On Fri, 26 Jan 2018, marxin at gcc dot gnu.org wrote:
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84044
>
> --- Comment #5 from Martin Liška ---
> (In reply to Richard Biener from comment
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84044
--- Comment #6 from Jan Hubicka ---
> > @@ -5150,15 +5145,9 @@ free_lang_data_in_type (tree type)
> > {
> > free_lang_data_in_binfo (TYPE_BINFO (type));
> > /* We need to preserve link to bases and virtual table for
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84044
--- Comment #5 from Martin Liška ---
(In reply to Richard Biener from comment #4)
> (In reply to Martin Liška from comment #3)
> > Fixed on trunk by Richi's r256685. Is it intentional Richi that the revision
> > should fix such situations?
>
>
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84044
--- Comment #4 from Richard Biener ---
(In reply to Martin Liška from comment #3)
> Fixed on trunk by Richi's r256685. Is it intentional Richi that the revision
> should fix such situations?
Not really. It means that the following hunk removes
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84044
Martin Liška changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
Last reconfirmed|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84044
--- Comment #2 from Geoffrey Allott ---
Or even simply
A a;
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84044
--- Comment #1 from Geoffrey Allott ---
I discovered that in b.cpp a free function
A get() {
return A();
}
also triggers the error. Struct B is not necessary.
16 matches
Mail list logo
