[testsuite] Fix PR93935 to guard case under vect_hw_misalign

Message ID 29d923f8-e217-c078-b6ec-7b3b6aded25a@linux.ibm.com
State New
Headers show
Series
  • [testsuite] Fix PR93935 to guard case under vect_hw_misalign
Related show

Commit Message

Kewen.Lin Feb. 26, 2020, 6:17 a.m.
Hi,

This patch is to apply the same fix as r267528 to another similar case
bb-slp-over-widen-2.c which requires misaligned vector access.

Verified it on ppc64-redhat-linux (Power7 BE).

Is it ok for trunk?

BR,
Kewen
-----------

gcc/testsuite/ChangeLog

2020-02-26  Kewen Lin  <linkw@gcc.gnu.org>

	PR testsuite/93935
	* gcc.dg/vect/bb-slp-over-widen-2.c: Expect basic block vectorized
	messages only on vect_hw_misalign targets.

* patch *:

Comments

Iain Buclaw via Gcc-patches March 11, 2020, 6:22 a.m. | #1
Hi,

Gentle ping this patch, also request to backport to gcc9 after some burn-in time.

BR,
Kewen

on 2020/2/26 下午2:17, Kewen.Lin wrote:
> Hi,

> 

> This patch is to apply the same fix as r267528 to another similar case

> bb-slp-over-widen-2.c which requires misaligned vector access.

> 

> Verified it on ppc64-redhat-linux (Power7 BE).

> 

> Is it ok for trunk?

> 

> BR,

> Kewen

> -----------

> 

> gcc/testsuite/ChangeLog

> 

> 2020-02-26  Kewen Lin  <linkw@gcc.gnu.org>

> 

> 	PR testsuite/93935

> 	* gcc.dg/vect/bb-slp-over-widen-2.c: Expect basic block vectorized

> 	messages only on vect_hw_misalign targets.

> 

> * patch *:

> 

> diff --git a/gcc/testsuite/gcc.dg/vect/bb-slp-over-widen-2.c b/gcc/testsuite/gcc.dg/vect/bb-slp-over-widen-2.c

> index 3750fb7..042b7e9 100644

> --- a/gcc/testsuite/gcc.dg/vect/bb-slp-over-widen-2.c

> +++ b/gcc/testsuite/gcc.dg/vect/bb-slp-over-widen-2.c

> @@ -63,4 +63,4 @@ main (void)

>  /* { dg-final { scan-tree-dump "demoting int to signed short" "slp2" { target { ! vect_widen_shift } } } } */

>  /* { dg-final { scan-tree-dump "demoting int to unsigned short" "slp2" { target { ! vect_widen_shift } } } } */

>  /* { dg-final { scan-tree-dump {\.AVG_FLOOR} "slp2" { target vect_avg_qi } } } */

> -/* { dg-final { scan-tree-dump-times "basic block vectorized" 2 "slp2" } } */

> +/* { dg-final { scan-tree-dump-times "basic block vectorized" 2 "slp2" { target vect_hw_misalign } } } */

>
Iain Buclaw via Gcc-patches March 12, 2020, 4:06 a.m. | #2
On Wed, 2020-03-11 at 14:22 +0800, Kewen.Lin via Gcc-patches wrote:
> Hi,

> 

> Gentle ping this patch, also request to backport to gcc9 after some burn-in

> time.

> 

> BR,

> Kewen

> 

> on 2020/2/26 下午2:17, Kewen.Lin wrote:

> > Hi,

> > 

> > This patch is to apply the same fix as r267528 to another similar case

> > bb-slp-over-widen-2.c which requires misaligned vector access.

> > 

> > Verified it on ppc64-redhat-linux (Power7 BE).

> > 

> > Is it ok for trunk?

> > 

> > BR,

> > Kewen

> > -----------

> > 

> > gcc/testsuite/ChangeLog

> > 

> > 2020-02-26  Kewen Lin  <linkw@gcc.gnu.org>

> > 

> > 	PR testsuite/93935

> > 	* gcc.dg/vect/bb-slp-over-widen-2.c: Expect basic block vectorized

> > 	messages only on vect_hw_misalign targets.

OK for the trunk and gcc-9.

jeff
> >

Patch

diff --git a/gcc/testsuite/gcc.dg/vect/bb-slp-over-widen-2.c b/gcc/testsuite/gcc.dg/vect/bb-slp-over-widen-2.c
index 3750fb7..042b7e9 100644
--- a/gcc/testsuite/gcc.dg/vect/bb-slp-over-widen-2.c
+++ b/gcc/testsuite/gcc.dg/vect/bb-slp-over-widen-2.c
@@ -63,4 +63,4 @@  main (void)
 /* { dg-final { scan-tree-dump "demoting int to signed short" "slp2" { target { ! vect_widen_shift } } } } */
 /* { dg-final { scan-tree-dump "demoting int to unsigned short" "slp2" { target { ! vect_widen_shift } } } } */
 /* { dg-final { scan-tree-dump {\.AVG_FLOOR} "slp2" { target vect_avg_qi } } } */
-/* { dg-final { scan-tree-dump-times "basic block vectorized" 2 "slp2" } } */
+/* { dg-final { scan-tree-dump-times "basic block vectorized" 2 "slp2" { target vect_hw_misalign } } } */