From patchwork Sun Aug 29 06:28:55 2010
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Subject: pr43430-1.c requires vect_condition target
Date: Sat, 28 Aug 2010 20:28:55 -0000
From: Ira Rosen <IRAR@il.ibm.com>
X-Patchwork-Id: 62931
Message-Id: <OFFD3A5243.8535E8B8-ONC225778E.0022F305-C225778E.00239B5B@il.ibm.com>
To: "Bingfeng Mei" <bmei@broadcom.com>
Cc: "gcc-patches@gcc.gnu.org" <gcc-patches@gcc.gnu.org>

gcc-patches-owner@gcc.gnu.org wrote on 27/08/2010 03:13:06 PM:

>
> Hi,
> I noticed pr43430-1.c contains conditional code inside the loop.
> To vectorize the loop, vect_condition target is required. OK for
> trunk?
>

It's fine with me, but it maybe better to check for vect_condition when
checking if the loop got vectorized:

 /* { dg-final { cleanup-tree-dump "vect" } } */


Thanks,
Ira

> Cheers,
> Bingfeng
>
> 2010-08-27  Bingfeng Mei  <bmei@broadcom.com>
>
>    * gcc.dg/vect/pr43430-1.c: Requires vect_condition target.
>
> Index: pr43430-1.c
> ===================================================================
> --- pr43430-1.c (revision 163583)
> +++ pr43430-1.c (working copy)
> @@ -1,4 +1,5 @@
>  /* { dg-require-effective-target vect_int } */
> +/* { dg-require-effective-target vect_condition } */
>
>  #include <stdarg.h>
>  #include "tree-vect.h"
>


Index: pr43430-1.c
===================================================================
--- pr43430-1.c (revision 163538)
+++ pr43430-1.c (working copy)
@@ -35,5 +35,5 @@
   return foo (data_ch1, data_ch2, 1);
 }

-/* { dg-final { scan-tree-dump-times "vectorized 1 loops" 1 "vect" } } */
+/* { dg-final { scan-tree-dump-times "vectorized 1 loops" 1
"vect" { target vect_condition } } } */