[Bug lto/69003] [4.9/5/6 Regression] Undefined reference with gcc -r incremental linking

[hubicka at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69003

--- Comment #7 from Jan Hubicka  ---
Author: hubicka
Date: Mon Jan 18 15:58:06 2016
New Revision: 232525

URL: https://gcc.gnu.org/viewcvs?rev=232525=gcc=rev
Log:
PR lto/69003
* lto-partition.c (rename_statics): Fix pasto.

Modified:
trunk/gcc/lto/ChangeLog
trunk/gcc/lto/lto-partition.c

[Bug lto/69003] [4.9/5/6 Regression] Undefined reference with gcc -r incremental linking

[sirl at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69003

--- Comment #6 from Franz Sirl  ---
Yes, this fixes the testcase and also the real application it was derived from
here. No testsuite regressions on x86_64 either.

[Bug lto/69003] [4.9/5/6 Regression] Undefined reference with gcc -r incremental linking

[hubicka at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69003

--- Comment #5 from Jan Hubicka  ---
Looks like a pasto in rename_statics.  I am testing:
Index: lto/lto-partition.c
===
--- lto/lto-partition.c (revision 232466)
+++ lto/lto-partition.c (working copy)
@@ -1077,8 +1077,8 @@ rename_statics (lto_symtab_encoder_t enc
  IDENTIFIER_POINTER
(DECL_ASSEMBLER_NAME (s->get_alias_target()->decl
&& ((s->real_symbol_p ()
- && !DECL_EXTERNAL (node->decl)
-&& !TREE_PUBLIC (node->decl))
+ && !DECL_EXTERNAL (s->decl)
+&& !TREE_PUBLIC (s->decl))
|| may_need_named_section_p (encoder, s))
&& (!encoder
|| lto_symtab_encoder_lookup (encoder, s) != LCC_NOT_FOUND))

[Bug lto/69003] [4.9/5/6 Regression] Undefined reference with gcc -r incremental linking

[hubicka at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69003

Jan Hubicka  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
   Assignee|unassigned at gcc dot gnu.org  |hubicka at gcc dot 
gnu.org

--- Comment #4 from Jan Hubicka  ---
incremental linking is supposed to work in GCC 6. I will take a look.

[Bug lto/69003] [4.9/5/6 Regression] Undefined reference with gcc -r incremental linking

[rguenth at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69003

Richard Biener  changed:

   What|Removed |Added

   Keywords||lto
   Priority|P3  |P2
 CC||hubicka at gcc dot gnu.org
  Known to work||4.8.3

--- Comment #3 from Richard Biener  ---
I've learned that partial linking isn't really supported with LTO.  Not sure
which part of the patches that try to implement it were applied for GCC 6 and
what is expected to work now.

[Bug lto/69003] [4.9/5/6 Regression] Undefined reference with gcc -r incremental linking

[pinskia at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69003

Andrew Pinski  changed:

   What|Removed |Added

   Keywords||link-failure, wrong-code
   Target Milestone|--- |4.9.4