On 08/12/14 18:45, David Malcolm wrote:
On Tue, 2014-08-12 at 17:15 -0400, Trevor Saunders wrote:
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> *.
I haven't updated my tree recently, so I don't have those in my tree yet :-)


Jeff

Reply via email to