[4/10] Don't assume vect_multiple_sizes means 2 sizes

Message ID	87zi83pc7o.fsf@linaro.org
State	New
Headers	show Delivered-To: patch@linaro.org Received-SPF: pass (google.com: domain of gcc-patches-return-465874-patch=linaro.org@gcc.gnu.org designates 209.132.180.131 as permitted sender) client-ip=209.132.180.131; DomainKey-Signature: a=rsa-sha1; c=nofws; d=gcc.gnu.org; h=list-id :list-unsubscribe:list-archive:list-post:list-help:sender:from :to:subject:references:date:in-reply-to:message-id:mime-version :content-type; q=dns; s=default; b=dlTz+2rC/tyH4Jkqqs6zGXbK2Mrwx DyEQVuDukSjbdBAxw7lQailBDZLdXJ0hmDuv8+60t9f/tMHKQTjrkOFJw2cZzMKo O0hrpa6VoPIUkoXYKeiE+//uQurN/4CfGVlGieBHpNBI7RzUe28mmAFmtgs2xfYp E0/7VmVqtwU070= Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk Sender: gcc-patches-owner@gcc.gnu.org From: Richard Sandiford <richard.sandiford@linaro.org> To: gcc-patches@gcc.gnu.org Mail-Followup-To: gcc-patches@gcc.gnu.org, richard.sandiford@linaro.org Subject: [4/10] Don't assume vect_multiple_sizes means 2 sizes References: <87inerqqyz.fsf@linaro.org> Date: Fri, 03 Nov 2017 16:18:51 +0000 In-Reply-To: <87inerqqyz.fsf@linaro.org> (Richard Sandiford's message of "Fri, 03 Nov 2017 16:14:44 +0000") Message-ID: <87zi83pc7o.fsf@linaro.org> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/25.2 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain
Series	Vectoriser testsuite tweaks \| expand [0/10] Vectoriser testsuite tweaks [1/10] Consistently use asm volatile ("" ::: "memory") in vect tests [2/10] Add VECTOR_BITS to tree-vect.h [3/10] Add available_vector_sizes to target-supports.exp [4/10] Don't assume vect_multiple_sizes means 2 sizes [5/10] Add vect_perm3_* target selectors [6/10] Add a vect_element_align_preferred target selector [7/10] Add a vect_unaligned_possible target selector [8/10] Add a vect_variable_length target selector [9/10] Add a vect_align_stack_vars target selector [10/10] Add a vect_masked_store target selector

Message ID

87zi83pc7o.fsf@linaro.org

State

New

Headers

Received-SPF: pass (google.com: domain of
	gcc-patches-return-465874-patch=linaro.org@gcc.gnu.org
	designates 209.132.180.131 as permitted sender)
	client-ip=209.132.180.131; 
DomainKey-Signature: a=rsa-sha1; c=nofws; d=gcc.gnu.org; h=list-id
	:list-unsubscribe:list-archive:list-post:list-help:sender:from
	:to:subject:references:date:in-reply-to:message-id:mime-version
	:content-type; q=dns; s=default; b=dlTz+2rC/tyH4Jkqqs6zGXbK2Mrwx
	DyEQVuDukSjbdBAxw7lQailBDZLdXJ0hmDuv8+60t9f/tMHKQTjrkOFJw2cZzMKo
	O0hrpa6VoPIUkoXYKeiE+//uQurN/4CfGVlGieBHpNBI7RzUe28mmAFmtgs2xfYp
	E0/7VmVqtwU070=
Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm
Precedence: bulk
Sender: gcc-patches-owner@gcc.gnu.org
From: Richard Sandiford <richard.sandiford@linaro.org>
To: gcc-patches@gcc.gnu.org
Mail-Followup-To: gcc-patches@gcc.gnu.org, richard.sandiford@linaro.org
Subject: [4/10] Don't assume vect_multiple_sizes means 2 sizes
References: <87inerqqyz.fsf@linaro.org>
Date: Fri, 03 Nov 2017 16:18:51 +0000
In-Reply-To: <87inerqqyz.fsf@linaro.org> (Richard Sandiford's message of
	"Fri, 03 Nov 2017 16:14:44 +0000")
Message-ID: <87zi83pc7o.fsf@linaro.org>
User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/25.2 (gnu/linux)
MIME-Version: 1.0
Content-Type: text/plain

Series

Vectoriser testsuite tweaks | expand

Commit Message

Richard Sandiford Nov. 3, 2017, 4:18 p.m. UTC

Some tests assumed that there would only be 2 vector sizes if
vect_multiple_sizes, whereas for SVE there are three (SVE, 128-bit
and 64-bit).  This patch replaces scan-tree-dump-times with
scan-tree-dump for vect_multiple_sizes but keeps it for
!vect_multiple_sizes.


2017-11-03  Richard Sandiford  <richard.sandiford@linaro.org>
	    Alan Hayward  <alan.hayward@arm.com>
	    David Sherwood  <david.sherwood@arm.com>

gcc/testsuite/
	* gcc.dg/vect/no-vfa-vect-101.c: Use scan-tree-dump rather than
	scan-tree-dump-times for vect_multiple_sizes.
	* gcc.dg/vect/no-vfa-vect-102.c: Likewise.
	* gcc.dg/vect/no-vfa-vect-102a.c: Likewise.
	* gcc.dg/vect/no-vfa-vect-37.c: Likewise.
	* gcc.dg/vect/no-vfa-vect-79.c: Likewise.
	* gcc.dg/vect/vect-104.c: Likewise.

Comments

Jeff Law Nov. 8, 2017, 7:12 p.m. UTC | #1

On 11/03/2017 10:18 AM, Richard Sandiford wrote:
> Some tests assumed that there would only be 2 vector sizes if

> vect_multiple_sizes, whereas for SVE there are three (SVE, 128-bit

> and 64-bit).  This patch replaces scan-tree-dump-times with

> scan-tree-dump for vect_multiple_sizes but keeps it for

> !vect_multiple_sizes.

> 

> 

> 2017-11-03  Richard Sandiford  <richard.sandiford@linaro.org>

> 	    Alan Hayward  <alan.hayward@arm.com>

> 	    David Sherwood  <david.sherwood@arm.com>

> 

> gcc/testsuite/

> 	* gcc.dg/vect/no-vfa-vect-101.c: Use scan-tree-dump rather than

> 	scan-tree-dump-times for vect_multiple_sizes.

> 	* gcc.dg/vect/no-vfa-vect-102.c: Likewise.

> 	* gcc.dg/vect/no-vfa-vect-102a.c: Likewise.

> 	* gcc.dg/vect/no-vfa-vect-37.c: Likewise.

> 	* gcc.dg/vect/no-vfa-vect-79.c: Likewise.

> 	* gcc.dg/vect/vect-104.c: Likewise.

OK.
jeff

Index: gcc/testsuite/gcc.dg/vect/no-vfa-vect-101.c
===================================================================
--- gcc/testsuite/gcc.dg/vect/no-vfa-vect-101.c	2015-06-02 23:53:38.000000000 +0100
+++ gcc/testsuite/gcc.dg/vect/no-vfa-vect-101.c	2017-11-03 16:06:16.141037152 +0000
@@ -46,5 +46,5 @@  int main (void)
 
 /* { dg-final { scan-tree-dump-times "vectorized 1 loops" 0 "vect" } } */
 /* { dg-final { scan-tree-dump-times "can't determine dependence" 1 "vect" { target { ! vect_multiple_sizes } } } } */
-/* { dg-final { scan-tree-dump-times "can't determine dependence" 2 "vect" { target vect_multiple_sizes } } } */
+/* { dg-final { scan-tree-dump "can't determine dependence" "vect" { target vect_multiple_sizes } } } */
 
Index: gcc/testsuite/gcc.dg/vect/no-vfa-vect-102.c
===================================================================
--- gcc/testsuite/gcc.dg/vect/no-vfa-vect-102.c	2017-11-03 16:06:03.052282173 +0000
+++ gcc/testsuite/gcc.dg/vect/no-vfa-vect-102.c	2017-11-03 16:06:16.141037152 +0000
@@ -51,5 +51,5 @@  int main (void)
 
 /* { dg-final { scan-tree-dump-times "vectorized 1 loops" 0 "vect" } } */
 /* { dg-final { scan-tree-dump-times "possible dependence between data-refs" 1 "vect" { target { ! vect_multiple_sizes } } } } */
-/* { dg-final { scan-tree-dump-times "possible dependence between data-refs" 2 "vect" { target vect_multiple_sizes } } } */
+/* { dg-final { scan-tree-dump "possible dependence between data-refs" "vect" { target vect_multiple_sizes } } } */
 
Index: gcc/testsuite/gcc.dg/vect/no-vfa-vect-102a.c
===================================================================
--- gcc/testsuite/gcc.dg/vect/no-vfa-vect-102a.c	2017-11-03 16:06:03.052282173 +0000
+++ gcc/testsuite/gcc.dg/vect/no-vfa-vect-102a.c	2017-11-03 16:06:16.141037152 +0000
@@ -51,5 +51,5 @@  int main (void)
 
 /* { dg-final { scan-tree-dump-times "vectorized 1 loops" 0 "vect" } } */
 /* { dg-final { scan-tree-dump-times "possible dependence between data-refs" 1 "vect" { target { ! vect_multiple_sizes } } } } */
-/* { dg-final { scan-tree-dump-times "possible dependence between data-refs" 2 "vect" { target vect_multiple_sizes } } } */
+/* { dg-final { scan-tree-dump "possible dependence between data-refs" "vect" { target vect_multiple_sizes } } } */
 
Index: gcc/testsuite/gcc.dg/vect/no-vfa-vect-37.c
===================================================================
--- gcc/testsuite/gcc.dg/vect/no-vfa-vect-37.c	2015-06-02 23:53:38.000000000 +0100
+++ gcc/testsuite/gcc.dg/vect/no-vfa-vect-37.c	2017-11-03 16:06:16.141037152 +0000
@@ -59,4 +59,4 @@  int main (void)
    prevent vectorization on some targets.  */
 /* { dg-final { scan-tree-dump-times "vectorized 2 loops" 1 "vect" { xfail *-*-* } } } */
 /* { dg-final { scan-tree-dump-times "can't determine dependence" 2 "vect" { target { ! vect_multiple_sizes } } } } */
-/* { dg-final { scan-tree-dump-times "can't determine dependence" 4 "vect" { target vect_multiple_sizes } } } */
+/* { dg-final { scan-tree-dump "can't determine dependence" "vect" { target vect_multiple_sizes } } } */
Index: gcc/testsuite/gcc.dg/vect/no-vfa-vect-79.c
===================================================================
--- gcc/testsuite/gcc.dg/vect/no-vfa-vect-79.c	2015-06-02 23:53:35.000000000 +0100
+++ gcc/testsuite/gcc.dg/vect/no-vfa-vect-79.c	2017-11-03 16:06:16.141037152 +0000
@@ -47,4 +47,4 @@  int main (void)
   prevent vectorization on some targets.  */
 /* { dg-final { scan-tree-dump-times "vectorized 1 loops" 1 "vect"  { xfail *-*-* } } } */
 /* { dg-final { scan-tree-dump-times "can't determine dependence" 1 "vect" { target { ! vect_multiple_sizes } } } } */
-/* { dg-final { scan-tree-dump-times "can't determine dependence" 2 "vect" { target vect_multiple_sizes } } } */
+/* { dg-final { scan-tree-dump "can't determine dependence" "vect" { target vect_multiple_sizes } } } */
Index: gcc/testsuite/gcc.dg/vect/vect-104.c
===================================================================
--- gcc/testsuite/gcc.dg/vect/vect-104.c	2017-11-03 16:06:03.054282499 +0000
+++ gcc/testsuite/gcc.dg/vect/vect-104.c	2017-11-03 16:06:16.142036841 +0000
@@ -63,5 +63,5 @@  int main (void)
 
 /* { dg-final { scan-tree-dump-times "vectorized 1 loops" 0 "vect" } } */
 /* { dg-final { scan-tree-dump-times "possible dependence between data-refs" 1 "vect" { target { ! vect_multiple_sizes } } } } */
-/* { dg-final { scan-tree-dump-times "possible dependence between data-refs" 2 "vect" { target vect_multiple_sizes } } } */
+/* { dg-final { scan-tree-dump "possible dependence between data-refs" "vect" { target vect_multiple_sizes } } } */

[4/10] Don't assume vect_multiple_sizes means 2 sizes

Commit Message

Comments

Patch