This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: tree profilnig merge: Multiple EH datastructuers


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~


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]