Re: [patch] Hash table changes from cxx-conversion branch - config part

2013-05-29 Thread Lawrence Crowl
On 5/21/13, Diego Novillo wrote: > On May 13, 2013 Lawrence Crowl wrote: > > I still have not heard from i386 or ia64 folks. Anyone? > > The i386 bits look fine to me as well. Please wait 48 hours to > give the i386 maintainers a chance to object. Committed. -- Lawrence Crowl

Re: [patch] Hash table changes from cxx-conversion branch - config part

2013-05-14 Thread Steve Ellcey
On Mon, 2013-05-13 at 15:03 -0700, Lawrence Crowl wrote: > I still have not heard from i386 or ia64 folks. Anyone? The IA64 part looks OK to me. Steve Ellcey sell...@imgtec.com (sell...@mips.com)

Re: [patch] Hash table changes from cxx-conversion branch - config part

2013-05-13 Thread Lawrence Crowl
I still have not heard from i386 or ia64 folks. Anyone? On 4/24/13, Lawrence Crowl wrote: > This patch is a consolodation of the hash_table patches to the > cxx-conversion branch for files under gcc/config. > > Recipients: > config/arm/arm.c - ni...@redhat.com, ramana.radhakrish...@arm.com > con

Re: [patch] Hash table changes from cxx-conversion branch - config part

2013-04-26 Thread Rainer Orth
Lawrence, > * config/sol2.c'solaris_comdat_htab > > Fold comdat_hash and comdat_eq into new struct comdat_entry_hasher. [...] > Index: gcc/ChangeLog > > 2013-04-24 Lawrence Crowl > * config/sol2.c (solaris_comdat_htab): > Change type to hash_table. Update dependent calls and types.

Re: [patch] Hash table changes from cxx-conversion branch

2013-04-25 Thread Lawrence Crowl
On 4/25/13, Diego Novillo wrote: > On 2013-04-24 15:58 , Lawrence Crowl wrote: >> * var-tracking.c'emit_note_data_def.vars >> * var-tracking.c'shared_hash_def.htab >> * var-tracking.c'changed_variables >> >> Fold variable_htab_hash, variable_htab_eq, variable_htab_free >>into new struct variab

Re: [patch] Hash table changes from cxx-conversion branch

2013-04-25 Thread Lawrence Crowl
On 4/25/13, Richard Biener wrote: > Thus, the patch is ok apart from the var-tracking.c bits which > I defer to respective maintainers. Okay, I split out the var-tracking.c changes. I've committed the rest to trunk. -- Lawrence Crowl

Re: [patch] Hash table changes from cxx-conversion branch

2013-04-25 Thread Diego Novillo
On 2013-04-24 15:58 , Lawrence Crowl wrote: * var-tracking.c'emit_note_data_def.vars * var-tracking.c'shared_hash_def.htab * var-tracking.c'changed_variables Fold variable_htab_hash, variable_htab_eq, variable_htab_free into new struct variable_hasher. Add typedef variable_table_type. Add typ

Re: [patch] Hash table changes from cxx-conversion branch - config part

2013-04-25 Thread Ramana Radhakrishnan
On 04/24/13 20:02, Lawrence Crowl wrote: This patch is a consolodation of the hash_table patches to the cxx-conversion branch for files under gcc/config. Recipients: config/arm/arm.c - ni...@redhat.com, ramana.radhakrish...@arm.com config/ia64/ia64.c - wil...@tuliptree.org, sell...@mips.com conf

Re: [patch] Hash table changes from cxx-conversion branch

2013-04-25 Thread Richard Biener
On Wed, Apr 24, 2013 at 9:58 PM, Lawrence Crowl wrote: > On 4/24/13, Lawrence Crowl wrote: >> Updated hash table patch. This patch does not include the config part, >> and does not include the entries already approved. >> >> The patch is attached due to size. > > Trying again. Patch attached co

Re: [patch] Hash table changes from cxx-conversion branch - config part

2013-04-24 Thread Richard Sandiford
Nice :-) MIPS parts look good apart from a minor formatting nit: Lawrence Crowl writes: > @@ -15945,7 +15958,8 @@ static int > mips_record_lo_sum (rtx *loc, void *data) > { >if (GET_CODE (*loc) == LO_SUM) > -mips_lo_sum_offset_lookup ((htab_t) data, XEXP (*loc, 1), INSERT); > +mips

Re: [patch] Hash table changes from cxx-conversion branch

2013-04-09 Thread Richard Biener
On Mon, Apr 8, 2013 at 11:45 PM, Lawrence Crowl wrote: > Ping? You didn't commit the ones I already approved? I don't want to go over them again ... Richard. > On 3/31/13, Lawrence Crowl wrote: >> On 3/28/13, Richard Biener wrote: >>> On Mar 27, 2013 Lawrence Crowl wrote: >>> > On 3/27/13,

Re: [patch] Hash table changes from cxx-conversion branch

2013-04-08 Thread Lawrence Crowl
Ping? On 3/31/13, Lawrence Crowl wrote: > On 3/28/13, Richard Biener wrote: >> On Mar 27, 2013 Lawrence Crowl wrote: >> > On 3/27/13, Richard Biener wrote: >> > > On Mar 23, 2013 Lawrence Crowl wrote: >> > > > This patch is a consolodation of the hash_table patches to >> > > > the cxx-convers

Re: [patch] Hash table changes from cxx-conversion branch

2013-03-28 Thread Richard Biener
On Wed, Mar 27, 2013 at 5:44 PM, Lawrence Crowl wrote: > On 3/27/13, Richard Biener wrote: >> On Mar 23, 2013 Lawrence Crowl wrote: >> > This patch is a consolodation of the hash_table patches to the >> > cxx-conversion branch. >> > >> > Update various hash tables from htab_t to hash_table. >> >

Re: [patch] Hash table changes from cxx-conversion branch

2013-03-27 Thread Lawrence Crowl
On 3/27/13, Richard Biener wrote: > On Mar 23, 2013 Lawrence Crowl wrote: > > This patch is a consolodation of the hash_table patches to the > > cxx-conversion branch. > > > > Update various hash tables from htab_t to hash_table. > > Modify types and calls to match. > > Ugh. Can you split it up

Re: [patch] Hash table changes from cxx-conversion branch

2013-03-27 Thread Martin Jambor
Hi, On Sat, Mar 23, 2013 at 03:36:35PM -0700, Lawrence Crowl wrote: > This patch is a consolodation of the hash_table patches to the > cxx-conversion branch. > I can't approve anything but I am fine with > * tree-sra.c (candidates): > Change type to hash_table. Update dependent cal

Re: [patch] Hash table changes from cxx-conversion branch

2013-03-27 Thread Richard Biener
On Sat, Mar 23, 2013 at 11:36 PM, Lawrence Crowl wrote: > This patch is a consolodation of the hash_table patches to the > cxx-conversion branch. > > Update various hash tables from htab_t to hash_table. > Modify types and calls to match. Ugh. Can you split it up somewhat ... like split target b