> On 2018-12-03 12:00 p.m., Michael Ploujnikov wrote: > > On 2018-12-01 11:29 a.m., H.J. Lu wrote: > >> This caused: > >> > >> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88297 > >> > > > > Sorry about that. Looks like I should have been testing with > > --with-build-config=bootstrap-lto rather than just --enable-bootstrap. > > > > The quick fix would be to undo the patch to create_virtual_clone or to > > just change clone_num_suffixes to key off of DECL_ASSEMBLER_NAME > > (node->decl) instead of node pointers. Any preferences? > > > > The harder fix would be to figure out why some nodes share the same > > names and fix that, but maybe that's just inevitable with LTO? > > > > - Michael > > > > Here's a quick fix while the issue is being investigated. > > Bootstrapped (--with-build-config=bootstrap-lto) and regtested on x86_64. > > Ok for trunk?
> From f5e2500f30ad337e85e0b53eaa15c724657966a2 Mon Sep 17 00:00:00 2001 > From: Michael Ploujnikov <michael.ploujni...@oracle.com> > Date: Mon, 3 Dec 2018 18:19:18 -0500 > Subject: [PATCH] PR ipa/88297 > > gcc/ChangeLog: > > 2018-12-03 Michael Ploujnikov <michael.ploujni...@oracle.com> > > PR ipa/88297 > * ipa-cp.c (create_specialized_node): Store clone counters by > node assembler names. > (ipcp_driver): Change type of clone_num_suffixes key type to > const char*. OK, thanks! Honza
