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