tree profilnig merge: Multiple EH datastructuers

Richard Henderson rth@redhat.com
Mon May 9 19:30:00 GMT 2005


On Tue, Apr 26, 2005 at 12:33:47AM +0200, Jan Hubicka wrote:
> + extern void set_eh_throw_stmt_table (struct function *, void *);
> + extern struct htab *get_eh_throw_stmt_table (struct function *);

Why "void *" on the one and and "struct htab *" on the other?
Seems like both could use "struct htab *", which would eliminate ...

> !   slot = htab_find_slot ((htab_t) get_eh_throw_stmt_table (cfun), n, INSERT);
...
> !   set_eh_throw_stmt_table (cfun, (void *) htab_create_ggc (31, struct_ptr_hash,
> ! 							   struct_ptr_eq,
> ! 							   ggc_free));

... these sorts of casts.


Otherwise ok.


r~



More information about the Gcc-patches mailing list