On Thu, Mar 28, 2019 at 11:02:52AM -0400, David Malcolm wrote:
> optrecord_json_writer::optinfo_to_json can in theory be called from any
> optimization pass, but currently uses get_fnname_from_decl, which
> is RTL-specific, which can lead to an ICE (PR middle-end/89725).

The ICE is a separate problem.  The problem with DECL_RTL early is
that it is undesirable to create RTL for decls during gimple passes,
we then need to make sure to clear it when versioning the function etc.
> 
> In that PR, Jakub suggested using either DECL_ASSEMBLER_NAME or the
> "printable name" (via current_function_name).
> 
> This patch makes it use DECL_ASSEMBLER_NAME.
> 
> Successfully bootstrapped & regrtested on x86_64-pc-linux-gnu.
> 
> OK for trunk?
> 
> gcc/ChangeLog:
>       PR middle-end/89725
>       * optinfo-emit-json.cc (optrecord_json_writer::optinfo_to_json):
>       Use DECL_ASSEMBLER_NAME rather than get_fnname_from_decl.

Ok, thanks.

        Jakub

Reply via email to