[Bug lto/47924] New: [4.6 Regression] Missed optimization with LTO

[Bug lto/47924] New: [4.6 Regression] Missed optimization with LTO

[d.g.gorbachev at gmail dot com]

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

           Summary: [4.6 Regression] Missed optimization with LTO
           Product: gcc
           Version: 4.6.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: lto
        AssignedTo: unassigned@gcc.gnu.org
        ReportedBy: d.g.gorbachev@gmail.com


Created attachment 23491
  --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=23491
Compile with -O2 -flto

[Bug lto/47924] [4.6 Regression] Missed optimization with LTO

[d.g.gorbachev at gmail dot com]

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

--- Comment #1 from Dmitry Gorbachev <d.g.gorbachev at gmail dot com> 2011-02-28 14:47:26 UTC ---
Created attachment 23492
  --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=23492
Generated asm

[Bug lto/47924] [4.6 Regression] Missed optimization with LTO

[d.g.gorbachev at gmail dot com]

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

--- Comment #2 from Dmitry Gorbachev <d.g.gorbachev at gmail dot com> 2011-02-28 14:48:01 UTC ---
Created attachment 23493
  --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=23493
Testcase #2

[Bug lto/47924] [4.6 Regression] Missed optimization with LTO

[rguenth at gcc dot gnu.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
   Last reconfirmed|                            |2011.02.28 15:19:15
         AssignedTo|unassigned at gcc dot       |rguenth at gcc dot gnu.org
                   |gnu.org                     |
   Target Milestone|---                         |4.6.0
     Ever Confirmed|0                           |1

--- Comment #3 from Richard Guenther <rguenth at gcc dot gnu.org> 2011-02-28 15:19:15 UTC ---
Confirmed.

I'll have a looksee.

[Bug lto/47924] [4.6 Regression] Missed optimization with LTO

[rguenth at gcc dot gnu.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED

--- Comment #4 from Richard Guenther <rguenth at gcc dot gnu.org> 2011-02-28 15:54:55 UTC ---
Hmm.  TYPE_CANONICAL seems to be broken (NULL) and we use alias-set 0 for both
int and short.  Probably because of

      /* Type merging will get confused by the canonical types as they
         are set by the middle-end.  */
      if (in_lto_p)
        TYPE_CANONICAL (node) = NULL_TREE;
      *nodep = node = gimple_register_type (node);

we are missing to register the canonical type here.

I have a patch.

[Bug lto/47924] [4.6 Regression] Missed optimization with LTO

[rguenth at gcc dot gnu.org]

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

--- Comment #5 from Richard Guenther <rguenth at gcc dot gnu.org> 2011-03-01 09:46:22 UTC ---
Author: rguenth
Date: Tue Mar  1 09:46:19 2011
New Revision: 170589

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=170589
Log:
2011-03-01  Richard Guenther  <rguenther@suse.de>

    PR lto/47924
    * lto-streamer.c (lto_record_common_node): Also register
    the canonical type.

    * gcc.dg/lto/pr47924_0.c: New testcase.

Added:
    trunk/gcc/testsuite/gcc.dg/lto/pr47924_0.c
Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/lto-streamer.c
    trunk/gcc/testsuite/ChangeLog

[Bug lto/47924] [4.6 Regression] Missed optimization with LTO

[rguenth at gcc dot gnu.org]

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

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

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

--- Comment #6 from Richard Guenther <rguenth at gcc dot gnu.org> 2011-03-01 09:46:57 UTC ---
Fixed.

[Bug lto/47924] [4.6 Regression] Missed optimization with LTO

[jamborm at gcc dot gnu.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |jamborm at gcc dot gnu.org

--- Comment #7 from Martin Jambor <jamborm at gcc dot gnu.org> 2011-03-02 00:45:15 UTC ---
*** Bug 47943 has been marked as a duplicate of this bug. ***