[Bug c/59825] New: [4.9 Regression] Many cilkplus test failures

[Bug c/59825] New: [4.9 Regression] Many cilkplus test failures

[hjl.tools at gmail dot com]

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

            Bug ID: 59825
           Summary: [4.9 Regression] Many cilkplus test failures
           Product: gcc
           Version: 4.9.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: c
          Assignee: unassigned at gcc dot gnu.org
          Reporter: hjl.tools at gmail dot com

On Linux/x86, revision 206621 gave:

FAIL: c-c++-common/cilk-plus/AN/rank_mismatch2.c  -fcilkplus -g -O0 -std=c99
(internal compiler error)
FAIL: c-c++-common/cilk-plus/AN/rank_mismatch2.c  -fcilkplus -g -O0 -std=c99
(test for excess errors)
FAIL: c-c++-common/cilk-plus/AN/rank_mismatch2.c  -fcilkplus (internal compiler
error)
FAIL: c-c++-common/cilk-plus/AN/rank_mismatch2.c  -fcilkplus -O2 -std=c99
(internal compiler error)
FAIL: c-c++-common/cilk-plus/AN/rank_mismatch2.c  -fcilkplus -O2 -std=c99 (test
for excess errors)
FAIL: c-c++-common/cilk-plus/AN/rank_mismatch2.c  -fcilkplus -O3 -std=c99
(internal compiler error)
FAIL: c-c++-common/cilk-plus/AN/rank_mismatch2.c  -fcilkplus -O3 -std=c99 (test
for excess errors)
FAIL: c-c++-common/cilk-plus/AN/rank_mismatch2.c  -fcilkplus -std=c99 (internal
compiler error)
FAIL: c-c++-common/cilk-plus/AN/rank_mismatch2.c  -fcilkplus -std=c99 (test for
excess errors)
FAIL: c-c++-common/cilk-plus/AN/rank_mismatch2.c  -fcilkplus (test for excess
errors)
FAIL: c-c++-common/cilk-plus/AN/rank_mismatch2.c  -g -fcilkplus (internal
compiler error)
FAIL: c-c++-common/cilk-plus/AN/rank_mismatch2.c  -g -fcilkplus (test for
excess errors)
FAIL: c-c++-common/cilk-plus/AN/rank_mismatch2.c  -g -O2 -ftree-vectorize
-fcilkplus (internal compiler error)
FAIL: c-c++-common/cilk-plus/AN/rank_mismatch2.c  -g -O2 -ftree-vectorize
-fcilkplus (test for excess errors)
FAIL: c-c++-common/cilk-plus/AN/rank_mismatch2.c  -O1 -fcilkplus (internal
compiler error)
FAIL: c-c++-common/cilk-plus/AN/rank_mismatch2.c  -O1 -fcilkplus (test for
excess errors)
FAIL: c-c++-common/cilk-plus/AN/rank_mismatch2.c  -O2 -fcilkplus (internal
compiler error)
FAIL: c-c++-common/cilk-plus/AN/rank_mismatch2.c  -O2 -fcilkplus (test for
excess errors)
FAIL: c-c++-common/cilk-plus/AN/rank_mismatch2.c  -O3 -fcilkplus (internal
compiler error)
FAIL: c-c++-common/cilk-plus/AN/rank_mismatch2.c  -O3 -fcilkplus (test for
excess errors)

Revision 206615 is OK.

[Bug c/59825] [4.9 Regression] Many cilkplus test failures

[hjl.tools at gmail dot com]

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

H.J. Lu <hjl.tools at gmail dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
   Last reconfirmed|                            |2014-01-15
                 CC|                            |jakub at gcc dot gnu.org
   Target Milestone|---                         |4.9.0
     Ever confirmed|0                           |1

--- Comment #1 from H.J. Lu <hjl.tools at gmail dot com> ---
It is caused by r206620.

[Bug c/59825] [4.9 Regression] Many cilkplus test failures

[jakub at gcc dot gnu.org]

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

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

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

--- Comment #2 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
That is known, I've said that explicitly in the patch submission that AN is
broken and will need to be fixed.

[Bug c/59825] Many cilkplus test failures

[law at redhat dot com]

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

Jeffrey A. Law <law at redhat dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |law at redhat dot com
           Assignee|unassigned at gcc dot gnu.org      |bviyer at gmail dot com
            Summary|[4.9 Regression] Many       |Many cilkplus test failures
                   |cilkplus test failures      |

--- Comment #3 from Jeffrey A. Law <law at redhat dot com> ---
Given Cilk+ is new in 4.9 the test failures are not a regression.

[Bug c/59825] Many cilkplus test failures

[hp at gcc dot gnu.org]

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

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

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

--- Comment #4 from Hans-Peter Nilsson <hp at gcc dot gnu.org> ---
(In reply to Jeffrey A. Law from comment #3)
> Given Cilk+ is new in 4.9 the test failures are not a regression.

Meh, I see you subscribe to the release-maintainer-definition; still they
passed before, so it's a regression on trunk, even if not compared to a
release.
(An autotester doesn't care whether the reason they passed is right or wrong,
it just keeps shouting at me...)  Anyway, apparently all targets see this, pain
dutily shared. Thanks to H.J. for the PR.

[Bug c/59825] Many cilkplus test failures

[law at redhat dot com]

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

--- Comment #5 from Jeffrey A. Law <law at redhat dot com> ---
It's certainly my mindset right now..  But fully understand the complaints
you're getting from the autotester :-)

[Bug c/59825] Many cilkplus test failures

[bviyer at gcc dot gnu.org]

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

--- Comment #6 from bviyer at gcc dot gnu.org ---
Author: bviyer
Date: Thu Jan 23 17:00:53 2014
New Revision: 206991

URL: http://gcc.gnu.org/viewcvs?rev=206991&root=gcc&view=rev
Log:
Fix for PR c/59825.
2014-01-23  Balaji V. Iyer  <balaji.v.iyer@intel.com>

        PR c/59825
        * c-array-notation.c (expand_array_notation_exprs): Rewrote this
        function to use walk_tree and moved a lot of its functionality to
        expand_array_notations.
        (expand_array_notations): New function.



Modified:
    trunk/gcc/c/ChangeLog
    trunk/gcc/c/c-array-notation.c

[Bug c/59825] Many cilkplus test failures

[law at redhat dot com]

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

Jeffrey A. Law <law at redhat dot com> changed:

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

--- Comment #7 from Jeffrey A. Law <law at redhat dot com> ---
Should be resolved by Balaji's trunk commit.