[Bug tree-optimization/46984] New: [4.6 Regression] g++.dg/torture/pr45699.C FAILs with -fno-early-inlining -flto

[Bug tree-optimization/46984] New: [4.6 Regression] g++.dg/torture/pr45699.C FAILs with -fno-early-inlining -flto

[zsojka at seznam dot cz]

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

           Summary: [4.6 Regression] g++.dg/torture/pr45699.C FAILs with
                    -fno-early-inlining -flto
           Product: gcc
           Version: 4.6.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: tree-optimization
        AssignedTo: unassigned@gcc.gnu.org
        ReportedBy: zsojka@seznam.cz
                CC: jamborm@gcc.gnu.org
              Host: x86_64-pc-linux-gnu
            Target: x86_64-pc-linux-gnu


+++ This bug was initially created as a clone of Bug #46053 +++

The fix for PR46053 doesn't work when -flto is applied:

Reduced testcase is the same:
http://gcc.gnu.org/bugzilla/attachment.cgi?id=22067

Output:
$ g++ -O -fipa-cp -fno-early-inlining -flto pr46053.C 
$ ./a.out 
Aborted

Tested revisions (for the reduced testcase):
r167954 - fail
4.5 r166509 - OK

[Bug tree-optimization/46984] [4.6 Regression] g++.dg/torture/pr45699.C FAILs with -fno-early-inlining -flto

[jamborm at gcc dot gnu.org]

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

Martin Jambor <jamborm at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |ASSIGNED
   Last reconfirmed|                            |2010.12.17 13:48:15
                 CC|                            |hubicka at gcc dot gnu.org
         AssignedTo|unassigned at gcc dot       |jamborm at gcc dot gnu.org
                   |gnu.org                     |
     Ever Confirmed|0                           |1

--- Comment #1 from Martin Jambor <jamborm at gcc dot gnu.org> 2010-12-17 13:48:15 UTC ---
I added thunk_delta streaming to analysis streaming and not decision
streaming, which of course cannot work.  Serves me right for not
testing lto.

I'll move the delta to struct cgraph_edge as a HOST_WIDE_INT.
Mine.

[Bug tree-optimization/46984] [4.6 Regression] g++.dg/torture/pr45699.C FAILs with -fno-early-inlining -flto

[jamborm at gcc dot gnu.org]

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

--- Comment #2 from Martin Jambor <jamborm at gcc dot gnu.org> 2010-12-21 15:11:50 UTC ---
Fix submitted to the mailing list:

http://gcc.gnu.org/ml/gcc-patches/2010-12/msg01618.html

[Bug tree-optimization/46984] [4.6 Regression] g++.dg/torture/pr45699.C FAILs with -fno-early-inlining -flto

[jakub at gcc dot gnu.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |jakub at gcc dot gnu.org
   Target Milestone|---                         |4.6.0

[Bug tree-optimization/46984] [4.6 Regression] g++.dg/torture/pr45699.C FAILs with -fno-early-inlining -flto

[jamborm at gcc dot gnu.org]

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

--- Comment #3 from Martin Jambor <jamborm at gcc dot gnu.org> 2011-01-03 13:06:58 UTC ---
Author: jamborm
Date: Mon Jan  3 13:06:54 2011
New Revision: 168420

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=168420
Log:
2011-01-03  Martin Jambor  <mjambor@suse.cz>

    PR tree-optimization/46984
    * cgraph.h (cgraph_indirect_call_info): make field thunk_delta
    HOST_WIDE_INT.
    (cgraph_create_indirect_edge): Fixed line length.
    (cgraph_indirect_call_info): Declare.
    (cgraph_make_edge_direct) Update declaration.
    * cgraph.c (cgraph_allocate_init_indirect_info): New function.
    (cgraph_create_indirect_edge): Use it.
    (cgraph_make_edge_direct): Made delta HOST_WIDE_INT.  Updated all
    callees.
    * cgraphunit.c (cgraph_redirect_edge_call_stmt_to_callee): Update for
    the new thunk_delta representation.
    * ipa-prop.c (ipa_make_edge_direct_to_target): Convert delta to
    HOST_WIDE_INT.
    (ipa_write_indirect_edge_info): Remove streaming of thunk_delta.
    (ipa_read_indirect_edge_info): Likewise.
    * lto-cgraph.c (output_edge_opt_summary): New function.
    (output_node_opt_summary): Call it on all outgoing edges.
    (input_edge_opt_summary): New function.
    (input_node_opt_summary): Call it on all outgoing edges.

    * testsuite/g++.dg/ipa/pr46984.C: New test.


Added:
    trunk/gcc/testsuite/g++.dg/ipa/pr46984.C
Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/cgraph.c
    trunk/gcc/cgraph.h
    trunk/gcc/cgraphunit.c
    trunk/gcc/ipa-prop.c
    trunk/gcc/lto-cgraph.c
    trunk/gcc/testsuite/ChangeLog

[Bug tree-optimization/46984] [4.6 Regression] g++.dg/torture/pr45699.C FAILs with -fno-early-inlining -flto

[jamborm at gcc dot gnu.org]

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

Martin Jambor <jamborm at gcc dot gnu.org> changed:

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

--- Comment #4 from Martin Jambor <jamborm at gcc dot gnu.org> 2011-01-03 13:08:54 UTC ---
Fixed.