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