> While at that, rework dump_cselib_table to not crash when
> cselib_preserved_hash_table is not allocated, and to remove the
> extraneous indirection from dump_cselib_val that made it inconvenient
> to call from a debugger.
> 
> Regstrapped on x86_64-linux-gnu.  Ok to install?
> 
> 
> for  gcc/ChangeLog
> 
>       * cselib.cc (cselib_invalidate_mem): Lookup the address as
>       part of canonicalizing it.
>       (dump_cselib_val): Split out of and rename to...
>       (dump_cselib_val_ptr): ... this.
>       (dump_cselib_table): Adjust.

The cselib_invalidate_mem change looks a bit risky at this stage if it does 
not really fix anything.  The rest is OK of course.

-- 
Eric Botcazou


Reply via email to