[Bug tree-optimization/58776] New: [4.9 Regression]: gcc.dg/tree-ssa/gen-vect-32.c scan-tree-dump-times vect "vectorized 1 loops" 1

[Bug tree-optimization/58776] New: [4.9 Regression]: gcc.dg/tree-ssa/gen-vect-32.c scan-tree-dump-times vect "vectorized 1 loops" 1

[hp at gcc dot gnu.org]

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

            Bug ID: 58776
           Summary: [4.9 Regression]: gcc.dg/tree-ssa/gen-vect-32.c
                    scan-tree-dump-times vect "vectorized 1 loops" 1
           Product: gcc
           Version: 4.7.0
            Status: UNCONFIRMED
          Keywords: missed-optimization
          Severity: normal
          Priority: P3
         Component: tree-optimization
          Assignee: unassigned at gcc dot gnu.org
          Reporter: hp at gcc dot gnu.org
                CC: davidxl at gcc dot gnu.org
              Host: x86_64-unknown-linux-gnu
            Target: cris-axis-elf

This test previously passed, now it fails.
A patch in the revision range (last_known_working:first_known_failing)
202971:202981 (most likely r202980)
exposed or caused this regression.  Since then it fails as follows:

Running /tmp/hpautotest-gcc1/gcc/gcc/testsuite/gcc.dg/tree-ssa/tree-ssa.exp ...
...
FAIL: gcc.dg/tree-ssa/gen-vect-32.c scan-tree-dump-times vect "vectorized 1
loops" 1

No hints from gcc.log.  Committer of r202980 CC:ed.

Adjustment like those in PR58556 and
<http://gcc.gnu.org/ml/gcc-patches/2013-10/msg01439.html> did not help; there
may be a larger issue here.

[Bug tree-optimization/58776] [4.9 Regression]: gcc.dg/tree-ssa/gen-vect-32.c scan-tree-dump-times vect "vectorized 1 loops" 1

[hp at gcc dot gnu.org]

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

--- Comment #1 from Hans-Peter Nilsson <hp at gcc dot gnu.org> ---
Created attachment 31029
  --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=31029&action=edit
gen-vect-32.c vect dump r203803 cris-elf

[Bug tree-optimization/58776] [4.9 Regression]: gcc.dg/tree-ssa/gen-vect-32.c scan-tree-dump-times vect "vectorized 1 loops" 1

[hp at gcc dot gnu.org]

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

--- Comment #2 from Hans-Peter Nilsson <hp at gcc dot gnu.org> ---
Created attachment 31030
  --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=31030&action=edit
Ditto generated assembly - no vectorizations done

[Bug tree-optimization/58776] [4.9 Regression]: gcc.dg/tree-ssa/gen-vect-32.c scan-tree-dump-times vect "vectorized 1 loops" 1

[rguenth at gcc dot gnu.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |4.9.0

[Bug tree-optimization/58776] [4.9 Regression]: gcc.dg/tree-ssa/gen-vect-32.c scan-tree-dump-times vect "vectorized 1 loops" 1

[rguenth at gcc dot gnu.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |WAITING
   Last reconfirmed|                            |2013-11-05
     Ever confirmed|0                           |1

--- Comment #3 from Richard Biener <rguenth at gcc dot gnu.org> ---
Does it work meanwhile?

[Bug tree-optimization/58776] [4.9 Regression]: gcc.dg/tree-ssa/gen-vect-32.c scan-tree-dump-times vect "vectorized 1 loops" 1

[hp at gcc dot gnu.org]

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

Hans-Peter Nilsson <hp at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|WAITING                     |NEW

--- Comment #4 from Hans-Peter Nilsson <hp at gcc dot gnu.org> ---
(In reply to Richard Biener from comment #3)
> Does it work meanwhile?

Short answer: no.

Longer answer:
Depends on the definition of "work".  Status is unchanged at r204406.
The test passes execution though.  From gcc.log:

PASS: gcc.dg/tree-ssa/gen-vect-32.c (test for excess errors)
PASS: gcc.dg/tree-ssa/gen-vect-32.c execution test
FAIL: gcc.dg/tree-ssa/gen-vect-32.c scan-tree-dump-times vect "vectorized 1
loops" 1
PASS: gcc.dg/tree-ssa/gen-vect-32.c scan-tree-dump-times vect "Vectorizing an
unaligned access" 0

[Bug tree-optimization/58776] [4.9 Regression]: gcc.dg/tree-ssa/gen-vect-32.c scan-tree-dump-times vect "vectorized 1 loops" 1

[rguenth at gcc dot gnu.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Priority|P3                          |P1
            Version|4.7.0                       |4.9.0

[Bug testsuite/58776] [4.9 Regression]: gcc.dg/tree-ssa/gen-vect-32.c scan-tree-dump-times vect "vectorized 1 loops" 1

[jakub at gcc dot gnu.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
           Assignee|unassigned at gcc dot gnu.org      |jakub at gcc dot gnu.org

--- Comment #5 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
Created attachment 31869
  --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=31869&action=edit
gcc49-pr58776.patch

For vectorization at -O2 the default now is the cheap cost model which doesn't
allow peeling for alignment, so what you get is IMHO desirable.

I think we should just add -fno-vect-cost-model to this test.

[Bug testsuite/58776] [4.9 Regression]: gcc.dg/tree-ssa/gen-vect-32.c scan-tree-dump-times vect "vectorized 1 loops" 1

[jakub at gcc dot gnu.org]

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

--- Comment #6 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
Author: jakub
Date: Fri Jan 17 14:43:37 2014
New Revision: 206712

URL: http://gcc.gnu.org/viewcvs?rev=206712&root=gcc&view=rev
Log:
    PR testsuite/58776
    * gcc.dg/tree-ssa-gen-vect-32.c: Add -fno-vect-cost-model to
    dg-options, use dg-additional-options for i?86/x86_64 to avoid
    option duplication.

Modified:
    trunk/gcc/testsuite/ChangeLog
    trunk/gcc/testsuite/gcc.dg/tree-ssa/gen-vect-32.c

[Bug testsuite/58776] [4.9 Regression]: gcc.dg/tree-ssa/gen-vect-32.c scan-tree-dump-times vect "vectorized 1 loops" 1

[jakub at gcc dot gnu.org]

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

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

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

--- Comment #7 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
Fixed.