Fix wrong code issue in ipa-devirt
Martin Jambor
mjambor@suse.cz
Tue Apr 8 16:28:00 GMT 2014
Hi,
On Tue, Apr 08, 2014 at 06:13:27AM +0200, Jan Hubicka wrote:
> Hi,
> while looking into devirtualization dumps, I noticed that I got one "false"
> typed out as "true". Not sure what I was thinking of.
>
> Martin, this may fix the wrong code issue you see in Firefox, hopefully.
it did not segfault the whole afternoon, so yes, I believe this is the
fix.
Thanks,
Martin
>
> Will commit it after x86_64-linux testing finishes tomorrow.
> Honza
>
> * ipa-devirt.c (maybe_record_node): Set completep to false
> instead of true when node is not recorded.
> Index: ipa-devirt.c
> ===================================================================
> --- ipa-devirt.c (revision 209170)
> +++ ipa-devirt.c (working copy)
> @@ -650,7 +650,7 @@ maybe_record_node (vec <cgraph_node *> &
> else if (completep
> && !type_in_anonymous_namespace_p
> (method_class_type (TREE_TYPE (target))))
> - *completep = true;
> + *completep = false;
> }
>
> /* See if BINFO's type match OUTER_TYPE. If so, lookup
More information about the Gcc-patches
mailing list