[Bug lto/44992] ld -r breaks LTO

[Bug lto/44992] ld -r breaks LTO

[jakub at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44992

Jakub Jelinek <jakub at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|4.6.0                       |4.6.1

--- Comment #9 from Jakub Jelinek <jakub at gcc dot gnu.org> 2011-03-25 19:53:20 UTC ---
GCC 4.6.0 is being released, adjusting target milestone.

[Bug lto/44992] ld -r breaks LTO

[rguenth at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44992

Richard Guenther <rguenth at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|4.6.1                       |---

[Bug lto/44992] ld -r breaks LTO

[andi-gcc at firstfloor dot org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44992

Andi Kleen <andi-gcc at firstfloor dot org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|REOPENED                    |RESOLVED
         Resolution|                            |FIXED

--- Comment #10 from Andi Kleen <andi-gcc at firstfloor dot org> 2011-10-07 05:42:50 UTC ---
I consider this fixed now because everything I need works together
with HJ's binutils.

Mainstream binutils will hopefull catch up eventually.

[Bug lto/44992] ld -r breaks LTO

[rguenth at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44992

--- Comment #11 from Richard Guenther <rguenth at gcc dot gnu.org> 2012-05-07 11:50:07 UTC ---
*** Bug 43576 has been marked as a duplicate of this bug. ***

[Bug lto/44992] ld -r breaks LTO

[andi-gcc at firstfloor dot org]

------- Comment #8 from andi-gcc at firstfloor dot org  2010-08-31 09:32 -------
Sorry this is not fixed yet, only partially. Still working on the last bits,
in particular passthrough of non LTOed code like assembler functions.


-- 

andi-gcc at firstfloor dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |REOPENED
         Resolution|FIXED                       |


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44992

[Bug lto/44992] ld -r breaks LTO

[rguenth at gcc dot gnu dot org]

------- Comment #7 from rguenth at gcc dot gnu dot org  2010-08-31 09:13 -------
Fixed.


-- 

rguenth at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED
   Target Milestone|---                         |4.6.0


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44992

[Bug lto/44992] ld -r breaks LTO

[ak at gcc dot gnu dot org]

------- Comment #6 from ak at gcc dot gnu dot org  2010-07-23 05:34 -------
Subject: Bug 44992

Author: ak
Date: Fri Jul 23 05:33:51 2010
New Revision: 162443

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=162443
Log:
gcc:

2010-07-10   Andi Kleen <ak@linux.intel.com>

        PR lto/44992
        * lto-opts.c (lto_write_options): Add NULL file_data argument to 
        lto_get_section_name.
        * lto-section-out.c (lto_destroy_simple_output_block): Likewise.
        * lto-streamer-out.c (produce_asm): Likewise.
        (copy_function): Likewise.
        (produce_symtab): Likewise.
        (produce_asm_for_decls): Likewise.
        * lto-streamer.c (lto_get_section_name): Add file_data argument.        
        Rewrite to add random postfix to LTO sections.
        * lto-streamer.h (lto_file_decl_data): Add next, id, resolutions.
        (lto_get_section_name): Add file_data argument to prototype.


lto:

2010-07-10   Andi Kleen <ak@linux.intel.com>

        PR lto/44992
        * lto.c: Include splay-tree.h
        (lto_resolution_read): Change to walk file_ids tree and parse
        extra file_id in resolution file.
        (lto_section_with_id): Add.
        (create_subid_section_table): Add.
        (lwstate): Add.
        (lto_create_files_from_ids): Add.
        (lto_file_read): Change to handle sub file ids and create list
        of file_datas. Add output argument for count.
        (get_section_data): Pass file_data to lto_get_section_name.
        (lto_flatten_file): Add.
        (read_cgraph_and_symbols): Handle linked lists of file_datas.

lto-plugin:

2010-07-10   Andi Kleen <ak@linux.intel.com>

        PR lto/44992
        * lto-plugin.c (sym_aux): Add.
        (plugin_symtab): Remove slots. Add aux and id.
        (parse_table_entry): Change to use aux instead of slots.
        (LTO_SECTION_PREFIX): Add.
        (translate): Improve buffer allocation. Change to append
        symbols to existing out buffer.
        (get_section): Remove.
        (process_symtab): Add. 
        (free_2): Free symtab->aux.
        (write_resolution): Handle aux instead of slots.
        Print sub id to resolution file.
        (claim_file_handler): Clear lto_file. Replace get_symtab/translate
        calls with call to process_symtab.  

Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/lto-opts.c
    trunk/gcc/lto-section-out.c
    trunk/gcc/lto-streamer-out.c
    trunk/gcc/lto-streamer.c
    trunk/gcc/lto-streamer.h
    trunk/gcc/lto/ChangeLog
    trunk/gcc/lto/lto.c
    trunk/lto-plugin/ChangeLog
    trunk/lto-plugin/lto-plugin.c


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44992

[Bug lto/44992] ld -r breaks LTO

[rguenth at gcc dot gnu dot org]

------- Comment #5 from rguenth at gcc dot gnu dot org  2010-07-20 09:00 -------
I was refering to a situation like

 gcc -c -flto t1.c
 gcc -c t2.c
 gcc -o t.o -r -nostdlib t1.o t2.o [-flto]
 gcc -o t t.o -flto

which would break with your solution (it's broken right now as well, of
course).

We could make it work by not emitting code but only LTO sections for t1.o.
Then the partial link would cause us to have both LTO sections and regular
sections, so at link-time we'd pick up the LTO parts for re-optimization
and link the code parts without re-optimization.  We'd still need a way
to merge LTO sections (or mangle them like you do), of course.


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44992

[Bug lto/44992] ld -r breaks LTO

[andi-gcc at firstfloor dot org]

------- Comment #4 from andi-gcc at firstfloor dot org  2010-07-19 19:46 -------
This is actually what I tried first, but it turned out to be quite complicated,
had to change a lot of code and my patch was growing and growing and it didn't
fit clearly with the different readers etc.

That is why I ended up with the post fixes instead which is much simpler
and isolated in a single part of the reader.

I don't understand the comment about section lengths? There's no dependency
on section lengths being a multiple of 4


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44992

[Bug lto/44992] ld -r breaks LTO

[jakub at gcc dot gnu dot org]

------- Comment #3 from jakub at gcc dot gnu dot org  2010-07-19 19:35 -------
I must say I don't like your solution.  IMHO much better is instead add a
header to LTO sections, which says the length of the LTO chunk (similarly e.g.
to how .debug_info section chunks have length in the header), perhaps version,
followed by that length of bytes of compressed stream, and change LTO reader to
iterate over all chunks in the LTO section.
This will also have advantage that it doesn't require non-ELF object formats to
have section length not a multiple of 4, etc.


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44992

[Bug lto/44992] ld -r breaks LTO

[andi-gcc at firstfloor dot org]

------- Comment #2 from andi-gcc at firstfloor dot org  2010-07-19 16:31 -------
Not sure I understand the comment.

The case I've been looking at is ld -r (without a LTO code generation stage)
to combine existing object and then using gold for the final linking/LTO code
generation based on the  combined objects.
For me it seems like gold handles this correctly (with my patches)

Your scenario sounds like ld -r with code generation? That's probably harder.


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44992

[Bug lto/44992] ld -r breaks LTO

[rguenth at gcc dot gnu dot org]

------- Comment #1 from rguenth at gcc dot gnu dot org  2010-07-19 16:20 -------
And partial linking support will break mixed LTO / non-LTO objects.  Unless
we drop all non-LTO sections from LTO objects and thus the .text sections
of partially linked mixed LTO / non-LTO objects will be still used in the
final link.


-- 

rguenth at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |rguenth at gcc dot gnu dot
                   |                            |org
             Status|UNCONFIRMED                 |NEW
     Ever Confirmed|0                           |1
           Keywords|                            |lto
   Last reconfirmed|0000-00-00 00:00:00         |2010-07-19 16:20:08
               date|                            |


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44992