[Bug middle-end/52656] New: [4.8 regression] gcc.target/sparc/fpmul-2.c FAILs

[Bug middle-end/52656] New: [4.8 regression] gcc.target/sparc/fpmul-2.c FAILs

[ro at gcc dot gnu.org]

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

             Bug #: 52656
           Summary: [4.8 regression] gcc.target/sparc/fpmul-2.c FAILs
    Classification: Unclassified
           Product: gcc
           Version: 4.8.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: middle-end
        AssignedTo: unassigned@gcc.gnu.org
        ReportedBy: ro@gcc.gnu.org
                CC: ebotcazou@gcc.gnu.org, rguenth@gcc.gnu.org
              Host: sparc-sun-solaris2*
            Target: sparc-sun-solaris2*
             Build: sparc-sun-solaris2*


As initially mentioned in PR testsuite/52614, gcc.target/sparc/fpmul-2.c has
recently started to fail:

FAIL: gcc.target/sparc/fpmul-2.c scan-tree-dump optimized "{ 2, 4, 6, 8 }"

It turned out that the failure is unrelated to the other ones, and a reghunt
revealed the culprit patch:

2012-03-16  Richard Guenther  <rguenther@suse.de>

        * tree.h (TREE_VECTOR_CST_ELTS): Remove.
        (VECTOR_CST_NELTS, VECTOR_CST_ELTS, VECTOR_CST_ELT): New defines.
        (struct tree_vector): Remove elements member, add variable size
        elts array member.
        (build_vector_stat): Declare.
        (build_vector): Define in terms of build_vector_stat.
[...]
        * config/sparc/sparc.c (sparc_handle_vis_mul8x16): Adjust interface
        and implementation.
        (sparc_fold_builtin): Adjust.

Here's the change in the optimized dumps:

> diff -u 23[45]/gcc/fpmul-2.c*|more
--- 234/gcc/fpmul-2.c.149t.optimized    2012-03-21 14:58:58.358234100 +0100
+++ 235/gcc/fpmul-2.c.149t.optimized    2012-03-21 16:18:28.973592400 +0100
@@ -48,7 +48,7 @@
 foo3 ()
 {
 <bb 2>:
-  return { 2, 4, 6, 8 };
+  return { 1, 2, 3, 4 };

 }

  Rainer

[Bug middle-end/52656] [4.8 regression] gcc.target/sparc/fpmul-2.c FAILs

[ebotcazou at gcc dot gnu.org]

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

Eric Botcazou <ebotcazou at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |ASSIGNED
   Last reconfirmed|                            |2012-03-21
                 CC|ebotcazou at gcc dot        |
                   |gnu.org                     |
         AssignedTo|unassigned at gcc dot       |ebotcazou at gcc dot
                   |gnu.org                     |gnu.org
     Ever Confirmed|0                           |1

--- Comment #1 from Eric Botcazou <ebotcazou at gcc dot gnu.org> 2012-03-21 15:49:46 UTC ---
Glad to see the code is properly covered by the testsuite.  Fixing...

[Bug middle-end/52656] [4.8 regression] gcc.target/sparc/fpmul-2.c FAILs

[rguenth at gcc dot gnu.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |4.8.0

[Bug middle-end/52656] [4.8 regression] gcc.target/sparc/fpmul-2.c FAILs

[ebotcazou at gcc dot gnu.org]

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

--- Comment #2 from Eric Botcazou <ebotcazou at gcc dot gnu.org> 2012-03-24 18:42:29 UTC ---
Author: ebotcazou
Date: Sat Mar 24 18:42:24 2012
New Revision: 185764

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=185764
Log:
    PR target/52656
    * config/sparc/sparc.c (sparc_handle_vis_mul8x16): Fix pasto.

Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/config/sparc/sparc.c

[Bug middle-end/52656] [4.8 regression] gcc.target/sparc/fpmul-2.c FAILs

[ebotcazou at gcc dot gnu.org]

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

Eric Botcazou <ebotcazou at gcc dot gnu.org> changed:

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

--- Comment #3 from Eric Botcazou <ebotcazou at gcc dot gnu.org> 2012-03-24 18:44:16 UTC ---
Thanks for reporting the problem.