On Tue, Aug 12, 2014 at 02:50:39PM -0600, Jeff Law wrote: > On 08/06/14 11:19, David Malcolm wrote: > >This gives a slight improvement in typesafety in cfgexpand.c > > > >gcc/ > > * cfgexpand.c (lab_rtx_for_bb): Convert from pointer_map_t to > > pointer_map<rtx>. > > (label_rtx_for_bb): Update for conversion of lab_rtx_for_bb to > > a pointer_map<rtx>, eliminating casts from void* to rtx. > > (expand_gimple_basic_block): Likewise. > > (pass_expand::execute): Likewise, using new/delete of > > pointer_map<rtx> rathern than pointer_map_create/destroy. NULLify > > the lab_rtx_for_bb ptr after deletion for good measure. > OK. I think this is still appropriate. It might even still apply > cleanly.
actually I suspect this patch is totally obsolete after my patches last week to remove pointer_map. This is now a hash_map<basic_block, rtx> *. sorry about the duplicated effort :/ Trev > > Seems like this could have gone forward independently of everything else. > > > jeff > >
