Re: [PATCH] Fix PR ipa/65908.

2015-06-21 Thread Jan Hubicka
Hi, this patch extends earlier Martin's patch. I completely removed parse_tree_args because it assumed that every function has only one set of parm types and one return value type. We really want to compare two sets of parameter types (DECL_ARGUMENTS and TYPE_PARM_TYPES) as they do not need to 100

Re: [PATCH] Fix PR ipa/65908.

2015-06-17 Thread Martin Liška
On 06/10/2015 01:50 PM, Martin Liška wrote: > On 05/15/2015 08:52 PM, Jan Hubicka wrote: >>> +/* Return true if DECL_ARGUMENT types are valid to be merged. */ >> Perhaps bettter as >> >> Perform additional check needed to match types function parameters that are >> used. Unlike for normal paramet

Re: [PATCH] Fix PR ipa/65908.

2015-06-10 Thread Martin Liška
se is not >relevant. > > Honza > Hi Honza. I forgot a bit about the patch. Please check updated version of the patch which can boostrap and survives regression tests. Martin >From c878090a910bafec5d1f85a58ec3c3dd5dfc9564 Mon Sep 17 00:00:00 2001 From: mliska Date: Fri, 15 M

Re: [PATCH] Fix PR ipa/65908.

2015-05-15 Thread Jan Hubicka
> +/* Return true if DECL_ARGUMENT types are valid to be merged. */ Perhaps bettter as Perform additional check needed to match types function parameters that are used. Unlike for normal parameters it matters if type is TYPE_RESTRICT and we make an assumption that REFERENCE_TYPE parameters are a

Re: [PATCH] Fix PR ipa/65908.

2015-05-15 Thread Martin Liška
rom 22ee6b7aaae73db83663e8b5c12054b2f126a53e Mon Sep 17 00:00:00 2001 From: mliska Date: Fri, 15 May 2015 13:23:33 +0200 Subject: [PATCH] Fix PR ipa/65908. gcc/testsuite/ChangeLog: 2015-05-12 Martin Liska * g++.dg/ipa/pr65908.C: New test. gcc/ChangeLog: 2015-05-12 Martin Liska PR

Re: [PATCH] Fix PR ipa/65908.

2015-05-15 Thread Jakub Jelinek
On Fri, May 15, 2015 at 10:38:57AM +0200, Martin Liška wrote: > Following patch is fix for GCC-5 branch for PR ipa/65908, was tested on > x86_64-linux-pc, as well as bootstrapped. > As soon as the patch is applied, I'm going to send the similar patch for > trunk. I'll leave the review to Honza o

[PATCH] Fix PR ipa/65908.

2015-05-15 Thread Martin Liška
Hello. Following patch is fix for GCC-5 branch for PR ipa/65908, was tested on x86_64-linux-pc, as well as bootstrapped. As soon as the patch is applied, I'm going to send the similar patch for trunk. Ready for 5 branch? Thanks, Martin gcc/testsuite/ChangeLog: 2015-05-12 Martin Liska