[Bug tree-optimization/21010] New: New gcc.dg/vect tests fail

[Bug tree-optimization/21010] New: New gcc.dg/vect tests fail

[jsm28 at gcc dot gnu dot org]

The failures

FAIL: gcc.dg/vect/vect-ifcvt-2.c scan-tree-dump-times vectorized 1 loops 1
FAIL: gcc.dg/vect/vect-ifcvt-3.c scan-tree-dump-times vectorized 1 loops 1
FAIL: gcc.dg/vect/vect-ifcvt-4.c scan-tree-dump-times vectorized 1 loops 1
FAIL: gcc.dg/vect/vect-ifcvt-5.c scan-tree-dump-times vectorized 1 loops 1
FAIL: gcc.dg/vect/vect-ifcvt-6.c scan-tree-dump-times vectorized 1 loops 1
FAIL: gcc.dg/vect/vect-ifcvt-7.c scan-tree-dump-times vectorized 1 loops 1
FAIL: gcc.dg/vect/vect-ifcvt-9.c scan-tree-dump-times vectorized 1 loops 2
FAIL: gcc.dg/vect/vect-none.c scan-tree-dump-times vectorized 0 loops 2
FAIL: gcc.dg/vect/vect-none.c scan-tree-dump-times vectorized 1 loops 1

(all new tests) appeared on mainline between 20050411 and 20050412, on
i686-pc-linux-gnu and ia64-hp-hpux11.23.  Perhaps these tests, added by

2004-04-11  Devang Patel  <dpatel@apple.com>

        * lib/target-supports.exp (check_effective_target_vect_condition): New.
        * gcc.dg/vect/vect-ifcvt-1.c: New test.
        * gcc.dg/vect/vect-ifcvt-2.c: New test.
        * gcc.dg/vect/vect-ifcvt-3.c: New test.
        * gcc.dg/vect/vect-ifcvt-4.c: New test.
        * gcc.dg/vect/vect-ifcvt-5.c: New test.
        * gcc.dg/vect/vect-ifcvt-6.c: New test.
        * gcc.dg/vect/vect-ifcvt-7.c: New test.
        * gcc.dg/vect/vect-none.c: Now one loop is vectorized.

are insufficiently portable.

-- 
           Summary: New gcc.dg/vect tests fail
           Product: gcc
           Version: 4.1.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P2
         Component: tree-optimization
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: jsm28 at gcc dot gnu dot org
                CC: dpatel at apple dot com,gcc-bugs at gcc dot gnu dot org


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

[Bug tree-optimization/21010] New gcc.dg/vect tests fail

[dpatel at apple dot com]

------- Additional Comments From dpatel at apple dot com  2005-04-13 22:47 -------
Subject: Re:  New: New gcc.dg/vect tests fail

But all of them require
     /* { dg-require-effective-target vect_condition } */
So, why they fail on other platforms ?
-
Devang



-- 


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

[Bug tree-optimization/21010] New gcc.dg/vect tests fail

[joseph at codesourcery dot com]

------- Additional Comments From joseph at codesourcery dot com  2005-04-14 00:32 -------
Subject: Re:  New gcc.dg/vect tests fail

On Wed, 13 Apr 2005, dpatel at apple dot com wrote:

> But all of them require
>      /* { dg-require-effective-target vect_condition } */
> So, why they fail on other platforms ?

1. They have that text before the dg-do line, which in accordance with the 
documentation in sourcebuild.texi is ineffective.

2. vect-none.c doesn't have that annotation at all (and it still fails).



-- 


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

[Bug testsuite/21010] New gcc.dg/vect tests fail

[pinskia at gcc dot gnu dot org]

------- Additional Comments From pinskia at gcc dot gnu dot org  2005-04-14 00:36 -------
Janis said she would take care of them: 
[17:22] < rth> janis: ping
[17:22] < janis> rth: pong
[17:23] < rth> janis: why is gcc.dg/vect/vect-ifcvt-2.c being tested on i386 even though 
vect_condition is only true for ppc?
[17:23] < janis> rth: no idea, I'll take a look
[17:24] < rth> janis: i assume devang did something wrong, but i can't see what.
[17:30] < janis> rth: you mean the test on the autovect branch? it's run for all targets, the options are 
overridden for powerpc, and the scan-tree-dump-times check is expected to fail on x86 and on 
x86_64
[17:36] < rth> janis: on, i mean on mainline.
[17:36] < janis> I didn't see that test on mainline, is it new today?
[17:37] < rth> janis: within the last day or three.
[17:40] < janis> rth: dg-do needs to come before directives that are local to the gcc testsuite
[17:41] < janis> rth: I'll fix those
[17:41] < rth> janis: thanks.
[17:41] < janis> np



-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |janis at gcc dot gnu dot org
             Status|UNCONFIRMED                 |NEW
          Component|tree-optimization           |testsuite
     Ever Confirmed|                            |1
   Last reconfirmed|0000-00-00 00:00:00         |2005-04-14 00:36:40
               date|                            |


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

[Bug testsuite/21010] New gcc.dg/vect tests fail

[dpatel at apple dot com]

------- Additional Comments From dpatel at apple dot com  2005-04-14 00:44 -------
Subject: Re:  New gcc.dg/vect tests fail

Thanks everyone!



-- 


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

[Bug testsuite/21010] New gcc.dg/vect tests fail

[janis at gcc dot gnu dot org]

------- Additional Comments From janis at gcc dot gnu dot org  2005-04-14 01:19 -------
There are additional problems with these tests and I don't have
time to finish looking at them today, but here's some info.
                                                                                
Autovectorization tests should never use 'dg-do run'.  There are
checks in vect.exp that determine whether dg-do-what is 'run' or
'compile' as the default based on whether the test hardware
supports vector instructions.  Rather than switching 'dg-do run'
and 'dg-require-*' I removed the 'dg-do run'.
                                                                                
Tests that generate temporary files should now clean up after
themselves; see the cleanup-dump-file directives in other tests.
                                                                                
The scan-dump checks for tests vect-ifcvt-[2345679].c fail on
powerpc64-linux for both -m32 and -m64; do they need additional
options?

-- 


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

[Bug testsuite/21010] New gcc.dg/vect tests fail

[cvs-commit at gcc dot gnu dot org]

------- Additional Comments From cvs-commit at gcc dot gnu dot org  2005-04-14 18:02 -------
Subject: Bug 21010

CVSROOT:	/cvs/gcc
Module name:	gcc
Changes by:	janis@gcc.gnu.org	2005-04-14 18:02:37

Modified files:
	gcc/testsuite/gcc.dg/vect: vect-ifcvt-1.c vect-ifcvt-2.c 
	                           vect-ifcvt-3.c vect-ifcvt-4.c 
	                           vect-ifcvt-5.c vect-ifcvt-6.c 
	                           vect-ifcvt-7.c vect-ifcvt-9.c 
	gcc/testsuite  : ChangeLog 

Log message:
	PR testsuite/21010
	* gcc.dg/vect/vect-ifcvt-1.c: Remove dg-do, add cleanup.
	* gcc.dg/vect/vect-ifcvt-2.c: Ditto.
	* gcc.dg/vect/vect-ifcvt-3.c: Ditto.
	* gcc.dg/vect/vect-ifcvt-4.c: Ditto.
	* gcc.dg/vect/vect-ifcvt-5.c: Ditto.
	* gcc.dg/vect/vect-ifcvt-6.c: Ditto.
	* gcc.dg/vect/vect-ifcvt-7.c: Ditto.
	* gcc.dg/vect/vect-ifcvt-9.c: Ditto.

Patches:
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/testsuite/gcc.dg/vect/vect-ifcvt-1.c.diff?cvsroot=gcc&r1=1.2&r2=1.3
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/testsuite/gcc.dg/vect/vect-ifcvt-2.c.diff?cvsroot=gcc&r1=1.2&r2=1.3
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/testsuite/gcc.dg/vect/vect-ifcvt-3.c.diff?cvsroot=gcc&r1=1.2&r2=1.3
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/testsuite/gcc.dg/vect/vect-ifcvt-4.c.diff?cvsroot=gcc&r1=1.2&r2=1.3
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/testsuite/gcc.dg/vect/vect-ifcvt-5.c.diff?cvsroot=gcc&r1=1.2&r2=1.3
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/testsuite/gcc.dg/vect/vect-ifcvt-6.c.diff?cvsroot=gcc&r1=1.2&r2=1.3
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/testsuite/gcc.dg/vect/vect-ifcvt-7.c.diff?cvsroot=gcc&r1=1.2&r2=1.3
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/testsuite/gcc.dg/vect/vect-ifcvt-9.c.diff?cvsroot=gcc&r1=1.2&r2=1.3
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/testsuite/ChangeLog.diff?cvsroot=gcc&r1=1.5349&r2=1.5350



-- 


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

[Bug testsuite/21010] New gcc.dg/vect tests fail

[janis at gcc dot gnu dot org]

------- Additional Comments From janis at gcc dot gnu dot org  2005-04-14 18:22 -------
Fixed with the patch.  Today the tests pass on powerpc64-linux.

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


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

[Bug testsuite/21010] New gcc.dg/vect tests fail

[pinskia at gcc dot gnu dot org]

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |4.1.0


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