Re: Handle CONSTRUCTOR in operand_equal_p

2015-10-22 Thread Jan Hubicka
> > But only via GENERIC folding I suppose. Yes, we don't value-number > aggregates and generally PRE (and DOM via excessive jump-threading) > is the only pass that remotely handles this kind of situation. I actually think it is tail merging unifying the code, but I did not really look too deep

Re: Handle CONSTRUCTOR in operand_equal_p

2015-10-22 Thread Richard Biener
On Wed, 21 Oct 2015, Jan Hubicka wrote: > Hi, > this is updated patch I am going to commit. As discussed, we also need to > match > non-empty CONSTRUCTOR of vectors, but those should never be having CONSTANT > flags > set, so they need care in the other path trhough operand_equal_p, so I will

Re: Handle CONSTRUCTOR in operand_equal_p

2015-10-21 Thread Jan Hubicka
Hi, this is updated patch I am going to commit. As discussed, we also need to match non-empty CONSTRUCTOR of vectors, but those should never be having CONSTANT flags set, so they need care in the other path trhough operand_equal_p, so I will post separate patch for that. I updated the comment pe

Re: Handle CONSTRUCTOR in operand_equal_p

2015-10-14 Thread Richard Biener
On October 15, 2015 1:55:09 AM GMT+02:00, Jan Hubicka wrote: >> On October 14, 2015 6:27:02 PM GMT+02:00, Jan Hubicka > wrote: >> >Hi, >> >this patch adds the CONSTRUCTOR case discussed while back. Only >empty >> >constructors are matched, as those are only appearing in gimple >> >operand. >> >I

Re: Handle CONSTRUCTOR in operand_equal_p

2015-10-14 Thread Jan Hubicka
> On October 14, 2015 6:27:02 PM GMT+02:00, Jan Hubicka wrote: > >Hi, > >this patch adds the CONSTRUCTOR case discussed while back. Only empty > >constructors are matched, as those are only appearing in gimple > >operand. > >I tested that during bootstrap about 7500 matches are for empty ctors. >

Re: Handle CONSTRUCTOR in operand_equal_p

2015-10-14 Thread Richard Biener
On October 14, 2015 6:27:02 PM GMT+02:00, Jan Hubicka wrote: >Hi, >this patch adds the CONSTRUCTOR case discussed while back. Only empty >constructors are matched, as those are only appearing in gimple >operand. >I tested that during bootstrap about 7500 matches are for empty ctors. >There are co

Re: Handle CONSTRUCTOR in operand_equal_p

2015-10-14 Thread Jan Hubicka
> On 10/14/2015 10:27 AM, Jan Hubicka wrote: > >Hi, > >this patch adds the CONSTRUCTOR case discussed while back. Only empty > >constructors are matched, as those are only appearing in gimple operand. > >I tested that during bootstrap about 7500 matches are for empty ctors. > >There are couple hun

Re: Handle CONSTRUCTOR in operand_equal_p

2015-10-14 Thread Jeff Law
On 10/14/2015 10:27 AM, Jan Hubicka wrote: Hi, this patch adds the CONSTRUCTOR case discussed while back. Only empty constructors are matched, as those are only appearing in gimple operand. I tested that during bootstrap about 7500 matches are for empty ctors. There are couple hundred for non-em