Re: [PATCH][7/6] Allow anonymous SSA names

2012-08-10 Thread Richard Guenther
On Fri, 10 Aug 2012, Jan Hubicka wrote: > > > > This converts most users of create_tmp_{var,reg} to use anonymous > > SSA names. To give you one more reason to look at 6/6 ;) > > > > Bootstrapped on x86_64-unknown-linux-gnu, testing in progress. > > Very cool. Thanks for the hard work. Did yo

Re: [PATCH][7/6] Allow anonymous SSA names

2012-08-10 Thread Richard Guenther
On Thu, 9 Aug 2012, Diego Novillo wrote: > On 12-08-09 09:20 , Richard Guenther wrote: > > > if (interm_type) > >{ > > /* Create a type conversion HALF_TYPE->INTERM_TYPE. */ > > ! tmp = create_tmp_reg (interm_type, NULL); > > ! new_oprnd = ma

Re: [PATCH][7/6] Allow anonymous SSA names

2012-08-09 Thread Jan Hubicka
> > This converts most users of create_tmp_{var,reg} to use anonymous > SSA names. To give you one more reason to look at 6/6 ;) > > Bootstrapped on x86_64-unknown-linux-gnu, testing in progress. Very cool. Thanks for the hard work. Did you have time to test the memory use effets? (I have to

Re: [PATCH][7/6] Allow anonymous SSA names

2012-08-09 Thread Diego Novillo
On 12-08-09 09:20 , Richard Guenther wrote: if (interm_type) { /* Create a type conversion HALF_TYPE->INTERM_TYPE. */ ! tmp = create_tmp_reg (interm_type, NULL); ! new_oprnd = make_ssa_name (tmp, NULL); new_stmt = gimple_build_as

Re: [PATCH][7/6] Allow anonymous SSA names

2012-08-09 Thread Richard Henderson
On 08/09/2012 06:20 AM, Richard Guenther wrote: > This converts most users of create_tmp_{var,reg} to use anonymous > SSA names. To give you one more reason to look at 6/6 ;) Wow, there's some really nice cleanups in there. r~