* Transparent alias suport part 11 (vtable rewritting fix)
@ 2015-12-09 19:42 Jan Hubicka
0 siblings, 0 replies; only message in thread
From: Jan Hubicka @ 2015-12-09 19:42 UTC (permalink / raw)
To: gcc-patches
Hi,
this patch fixes a thinko in vtable rewritting where instead of checking
references we was checking list of nodes referring to vtable. The loop
looks for the entries in vtable that can be replaced by local aliases,
so this bug probably disabled most of the transformations.
It reproduced as an ICE while we tried to load body of an transparent alias
during the bootstrap with decl merging disabled.
This is down to one ICE during the build I will fix next.
Bootstrapped/regtested x86_64-linux, commited.
Honza
PR ipa/61886
* ipa-visibility.c (function_and_variable_visibility): Fix vtable
rewritting guard.
Index: ipa-visibility.c
===================================================================
--- ipa-visibility.c (revision 231472)
+++ ipa-visibility.c (working copy)
@@ -720,7 +720,7 @@ function_and_variable_visibility (bool w
bool found = false;
/* See if there is something to update. */
- for (i = 0; vnode->iterate_referring (i, ref); i++)
+ for (i = 0; vnode->iterate_reference (i, ref); i++)
if (ref->use == IPA_REF_ADDR
&& can_replace_by_local_alias_in_vtable (ref->referred))
{
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2015-12-09 19:42 UTC | newest]
Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-12-09 19:42 Transparent alias suport part 11 (vtable rewritting fix) Jan Hubicka
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).