[Bug middle-end/37338] New: [4.4 Regression] gcc.dg/array-init-1.c

[Bug middle-end/37338] New: [4.4 Regression] gcc.dg/array-init-1.c

[hjl dot tools at gmail dot com]

On Linux/ia32, revision 139863 gave

FAIL: gcc.dg/array-init-1.c scan-assembler-not abcdefghi
FAIL: gcc.dg/array-init-1.c scan-assembler-times
7017280452245743464|7523094288207667809|6867666564636261|1684234849|64636261 2

revision 139848 is OK.


-- 
           Summary: [4.4 Regression] gcc.dg/array-init-1.c
           Product: gcc
           Version: 4.4.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: middle-end
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: hjl dot tools at gmail dot com


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

[Bug middle-end/37338] [4.4 Regression] gcc.dg/array-init-1.c

[hjl dot tools at gmail dot com]

------- Comment #1 from hjl dot tools at gmail dot com  2008-09-02 18:56 -------
Revision 139854 is OK and revision 139856 is bad.


-- 

hjl dot tools at gmail dot com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |jh at suse dot cz


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

[Bug middle-end/37338] [4.4 Regression] gcc.dg/array-init-1.c

[rguenth at gcc dot gnu dot org]

-- 

rguenth at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |4.4.0


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

[Bug middle-end/37338] [4.4 Regression] gcc.dg/array-init-1.c

[jakub at gcc dot gnu dot org]

------- Comment #2 from jakub at gcc dot gnu dot org  2008-09-09 17:50 -------
The problem here is that gimplify_init_constructor calls can_move_by_pieces,
and the default MOVE_BY_PIECES_P uses optimize_insn_for_speed_p:
#define MOVE_BY_PIECES_P(SIZE, ALIGN) \
  (move_by_pieces_ninsns (SIZE, ALIGN, MOVE_MAX_PIECES + 1) \
   < (unsigned int) MOVE_RATIO (optimize_insn_for_speed_p ()))

But during gimplification *crtl is cleared, so eventhough
optimize_function_for_speed_p (cfun) is true, crtl->maybe_hot_insn_p is false.

Should gimplify_init_constructor temporarily set crtl->maybe_hot_insn_p = true?


-- 


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

[Bug middle-end/37338] [4.4 Regression] gcc.dg/array-init-1.c

[jakub at gcc dot gnu dot org]

-- 

jakub at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|unassigned at gcc dot gnu   |jakub at gcc dot gnu dot org
                   |dot org                     |
             Status|UNCONFIRMED                 |ASSIGNED
     Ever Confirmed|0                           |1
   Last reconfirmed|0000-00-00 00:00:00         |2008-09-10 09:11:11
               date|                            |


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

[Bug middle-end/37338] [4.4 Regression] gcc.dg/array-init-1.c

[jakub at gcc dot gnu dot org]

------- Comment #4 from jakub at gcc dot gnu dot org  2008-09-10 21:09 -------
Fixed.


-- 

jakub at gcc dot gnu dot org changed:

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


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

[Bug middle-end/37338] [4.4 Regression] gcc.dg/array-init-1.c

[jakub at gcc dot gnu dot org]

------- Comment #3 from jakub at gcc dot gnu dot org  2008-09-10 21:09 -------
Subject: Bug 37338

Author: jakub
Date: Wed Sep 10 21:08:17 2008
New Revision: 140249

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=140249
Log:
        PR middle-end/37338
        * gimplify.c (gimplify_body): Call default_rtl_profile.

Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/gimplify.c


-- 


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