[Bug middle-end/37218] New: [4.4 Regression] Revision 139525 caused many SLP regressions

[Bug middle-end/37218] New: [4.4 Regression] Revision 139525 caused many SLP regressions

[hjl dot tools at gmail dot com]

On Linux/ia32, Revision 139525 caused

FAIL: gcc.dg/vect/slp-14.c scan-tree-dump-times vect "vectorized 1 loops" 1
FAIL: gcc.dg/vect/slp-14.c scan-tree-dump-times vect "vectorizing stmts using
SLP" 2
FAIL: gcc.dg/vect/slp-15.c scan-tree-dump-times vect "vectorized 1 loops" 1
FAIL: gcc.dg/vect/slp-15.c scan-tree-dump-times vect "vectorizing stmts using
SLP" 2
FAIL: gcc.dg/vect/slp-2.c scan-tree-dump-times vect "vectorized 4 loops" 1
FAIL: gcc.dg/vect/slp-2.c scan-tree-dump-times vect "vectorizing stmts using
SLP" 4
FAIL: gcc.dg/vect/slp-20.c scan-tree-dump-times vect "vectorized 2 loops" 1
FAIL: gcc.dg/vect/slp-20.c scan-tree-dump-times vect "vectorizing stmts using
SLP" 4
FAIL: gcc.dg/vect/slp-22.c scan-tree-dump-times vect "vectorized 2 loops" 1
FAIL: gcc.dg/vect/slp-22.c scan-tree-dump-times vect "vectorizing stmts using
SLP" 6
FAIL: gcc.dg/vect/slp-25.c scan-tree-dump-times vect "vectorized 2 loops" 1
FAIL: gcc.dg/vect/slp-25.c scan-tree-dump-times vect "Alignment of access
forced using peeling" 2
FAIL: gcc.dg/vect/slp-9.c scan-tree-dump-times vect "vectorized 1 loops" 1
FAIL: gcc.dg/vect/slp-9.c scan-tree-dump-times vect "vectorizing stmts using
SLP" 1


-- 
           Summary: [4.4 Regression] Revision 139525 caused many SLP
                    regressions
           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=37218

[Bug middle-end/37218] [4.4 Regression] Revision 139525 caused many SLP regressions

[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=37218

[Bug middle-end/37218] [4.4 Regression] Revision 139525 caused many SLP regressions

[rguenth at gcc dot gnu dot org]

------- Comment #1 from rguenth at gcc dot gnu dot org  2008-08-24 08:39 -------
Well, it seems that revision included enabling ipa-cp by default at -O2 ;)

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

        * ipa-cp.c (constant_val_insert): Remove.
        (ipcp_propagate_one_const): Remove.
        (ipcp_create_replace_map): Always insert replacements to the map.
        (ipcp_insert_stage): Do not try to insert statements by hand.
        * tree-inline.c (insert_init_stmt): Break out from ...
        (setup_one_parameter): ... here; allow NULL BB pointer.
        (tree_function_versioning): Use setup_one_parameter to process
        replacement map.

Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/ipa-cp.c
    trunk/gcc/opts.c
           ^^^^^^^^^^
    trunk/gcc/tree-inline.c


of course most of the vect tests are really written without optimizers
in mind ...


-- 

rguenth at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
     Ever Confirmed|0                           |1
   Last reconfirmed|0000-00-00 00:00:00         |2008-08-24 08:39:43
               date|                            |


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

[Bug middle-end/37218] [4.4 Regression] Revision 139525 caused many SLP regressions

[dominiq at lps dot ens dot fr]

------- Comment #2 from dominiq at lps dot ens dot fr  2008-08-24 09:17 -------
> Well, it seems that revision included enabling ipa-cp by default at -O2 ;)

May this explain the following failures (i686-apple-darwin9):

[ibook-dhum] f90/bug% gfc -O2
/opt/gcc/_gcc_clean/gcc/testsuite/gfortran.dg/assumed_charlen_arg_1.f90
/opt/gcc/_gcc_clean/gcc/testsuite/gfortran.dg/assumed_charlen_arg_1.f90: In
function 'T.26':
/opt/gcc/_gcc_clean/gcc/testsuite/gfortran.dg/assumed_charlen_arg_1.f90:3:
internal compiler error: vector VEC(ipa_edge_args_t,base) index domain error,
in ipa_edge_removal_hook at ipa-prop.c:1060
[ibook-dhum] f90/bug% gfc -O2
/opt/gcc/_gcc_clean/gcc/testsuite/gfortran.dg/namelist_14.f90
/opt/gcc/_gcc_clean/gcc/testsuite/gfortran.dg/namelist_14.f90: In function
'T.92':
/opt/gcc/_gcc_clean/gcc/testsuite/gfortran.dg/namelist_14.f90:26: internal
compiler error: in referenced_var_lookup, at tree-dfa.c:563
[ibook-dhum] f90/bug% gfc -O2
/opt/gcc/_gcc_clean/gcc/testsuite/gfortran.dg/substr_4.f
/opt/gcc/_gcc_clean/gcc/testsuite/gfortran.dg/substr_4.f: In function 'T.39':
/opt/gcc/_gcc_clean/gcc/testsuite/gfortran.dg/substr_4.f:42: internal compiler
error: in referenced_var_lookup, at tree-dfa.c:563

or should I open another PR?


-- 


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

[Bug middle-end/37218] [4.4 Regression] Revision 139525 caused many SLP regressions

[jh at suse dot cz]

------- Comment #3 from jh at suse dot cz  2008-08-24 11:37 -------
Subject: Re:  [4.4 Regression] Revision 139525 caused many SLP regressions

> 
> or should I open another PR?

I've reverted the accidental commit as well as fixed this problem, so I
hope it is all OK.

Honza


-- 


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

[Bug middle-end/37218] [4.4 Regression] Revision 139525 caused many SLP regressions

[hjl dot tools at gmail dot com]

------- Comment #4 from hjl dot tools at gmail dot com  2008-08-25 16:33 -------
Fixed by revision 139532.


-- 

hjl dot tools at gmail dot com changed:

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


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