[PATCH v3] Make function clone name numbering independent.
Jan Hubicka
hubicka@ucw.cz
Tue Dec 4 14:07:00 GMT 2018
> 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.ploujnikov@oracle.com>
> Date: Mon, 3 Dec 2018 18:19:18 -0500
> Subject: [PATCH] PR ipa/88297
>
> gcc/ChangeLog:
>
> 2018-12-03 Michael Ploujnikov <michael.ploujnikov@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
More information about the Gcc-patches
mailing list