public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
From: Richard Guenther <richard.guenther@gmail.com>
To: gcc-patches@gcc.gnu.org, patches@linaro.org,
	richard.sandiford@linaro.org
Subject: Re: [10/9] Add tests for stride-3 accesses
Date: Fri, 15 Apr 2011 12:45:00 -0000	[thread overview]
Message-ID: <BANLkTikRzbWSKrhrxCWf3wCbDO3-gy_Ykg@mail.gmail.com> (raw)
In-Reply-To: <g4ipujedxy.fsf@linaro.org>

On Tue, Apr 12, 2011 at 4:34 PM, Richard Sandiford
<richard.sandiford@linaro.org> wrote:
> This patch adds a test for stride-3 accesses.  I didn't add any
> particularly complicated cases because I think the testsuite already
> covers the interaction between the strided loads & stores and other
> operations pretty well.  Let me know if there's something I should
> add though.
>
> Tested on x86_64-linux-gnu and arm-linux-gnueabi.  OK to install?

Ok.

Thanks,
Richard.

> Richard
>
>
> gcc/testsuite/
>        * gcc.dg/vect/vect-strided-u16-i3.c: New test.
>
> Index: gcc/testsuite/gcc.dg/vect/vect-strided-u16-i3.c
> ===================================================================
> --- /dev/null   2011-03-23 08:42:11.268792848 +0000
> +++ gcc/testsuite/gcc.dg/vect/vect-strided-u16-i3.c     2011-04-12 11:55:17.000000000 +0100
> @@ -0,0 +1,112 @@
> +#include <stdarg.h>
> +#include "tree-vect.h"
> +
> +#define N 128
> +
> +typedef struct {
> +   unsigned short a;
> +   unsigned short b;
> +   unsigned short c;
> +} s;
> +
> +#define A(I) (I)
> +#define B(I) ((I) * 2)
> +#define C(I) ((unsigned short) ~((I) ^ 0x18))
> +
> +void __attribute__ ((noinline))
> +check1 (s *res)
> +{
> +  int i;
> +
> +  for (i = 0; i < N; i++)
> +    if (res[i].a != C (i)
> +       || res[i].b != A (i)
> +       || res[i].c != B (i))
> +      abort ();
> +}
> +
> +void __attribute__ ((noinline))
> +check2 (unsigned short *res)
> +{
> +  int i;
> +
> +  for (i = 0; i < N; i++)
> +    if (res[i] != (unsigned short) (A (i) + B (i) + C (i)))
> +      abort ();
> +}
> +
> +void __attribute__ ((noinline))
> +check3 (s *res)
> +{
> +  int i;
> +
> +  for (i = 0; i < N; i++)
> +    if (res[i].a != i
> +       || res[i].b != i
> +       || res[i].c != i)
> +      abort ();
> +}
> +
> +void __attribute__ ((noinline))
> +check4 (unsigned short *res)
> +{
> +  int i;
> +
> +  for (i = 0; i < N; i++)
> +    if (res[i] != (unsigned short) (A (i) + B (i)))
> +      abort ();
> +}
> +
> +void __attribute__ ((noinline))
> +main1 (s *arr)
> +{
> +  int i;
> +  s *ptr = arr;
> +  s res1[N];
> +  unsigned short res2[N];
> +
> +  for (i = 0; i < N; i++)
> +    {
> +      res1[i].a = arr[i].c;
> +      res1[i].b = arr[i].a;
> +      res1[i].c = arr[i].b;
> +    }
> +  check1 (res1);
> +
> +  for (i = 0; i < N; i++)
> +    res2[i] = arr[i].a + arr[i].b + arr[i].c;
> +  check2 (res2);
> +
> +  for (i = 0; i < N; i++)
> +    {
> +      res1[i].a = i;
> +      res1[i].b = i;
> +      res1[i].c = i;
> +    }
> +  check3 (res1);
> +
> +  for (i = 0; i < N; i++)
> +    res2[i] = arr[i].a + arr[i].b;
> +  check4 (res2);
> +}
> +
> +int main (void)
> +{
> +  int i;
> +  s arr[N];
> +
> +  check_vect ();
> +
> +  for (i = 0; i < N; i++)
> +    {
> +      arr[i].a = A (i);
> +      arr[i].b = B (i);
> +      arr[i].c = C (i);
> +    }
> +  main1 (arr);
> +
> +  return 0;
> +}
> +
> +/* { dg-final { scan-tree-dump-times "vectorized 4 loops" 1 "vect"  { target vect_strided3 } } } */
> +/* { dg-final { cleanup-tree-dump "vect" } } */
>

      reply	other threads:[~2011-04-15 12:44 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-04-12 13:21 [0/9] Direct support for loads and stores of interleaved vectors Richard Sandiford
2011-04-12 13:25 ` [1/9] Generalise vect_create_data_ref_ptr Richard Sandiford
2011-04-12 13:30   ` Richard Guenther
2011-04-12 13:28 ` [2/9] Reindent parts of vectorizable_load and vectorizable_store Richard Sandiford
2011-04-12 13:33   ` Richard Guenther
2011-04-12 14:39     ` Richard Sandiford
2011-04-12 13:40 ` [3/9] STMT_VINFO_RELATED_STMT handling in vectorizable_store Richard Sandiford
2011-04-17 10:25   ` Ira Rosen
2011-04-12 13:44 ` [4/9] Move power-of-two checks for interleaving Richard Sandiford
2011-04-12 13:57   ` Richard Guenther
2011-04-12 13:59 ` [5/9] Main target-independent support for direct interleaving Richard Sandiford
2011-04-17 14:26   ` Ira Rosen
2011-04-18 11:54   ` Richard Guenther
2011-04-18 11:57     ` Richard Sandiford
2011-04-18 12:54       ` Richard Guenther
2011-04-18 12:58         ` Richard Sandiford
2011-04-18 13:22           ` Richard Guenther
2011-04-12 14:01 ` [6/9] NEON vec_load_lanes and vec_store_lanes patterns Richard Sandiford
2011-04-15 13:20   ` Richard Earnshaw
2011-04-12 14:14 ` [7/9] Testsuite: remove vect_{extract_even_odd,strided}_wide Richard Sandiford
2011-04-15 12:43   ` Richard Guenther
2011-04-12 14:19 ` [8/9] Testsuite: split tests for strided accesses Richard Sandiford
2011-04-15 12:44   ` Richard Guenther
2011-04-12 14:29 ` [9/9] Testsuite: Replace vect_strided with vect_stridedN Richard Sandiford
2011-04-15 12:44   ` Richard Guenther
2011-04-12 14:34 ` [10/9] Add tests for stride-3 accesses Richard Sandiford
2011-04-15 12:45   ` Richard Guenther [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=BANLkTikRzbWSKrhrxCWf3wCbDO3-gy_Ykg@mail.gmail.com \
    --to=richard.guenther@gmail.com \
    --cc=gcc-patches@gcc.gnu.org \
    --cc=patches@linaro.org \
    --cc=richard.sandiford@linaro.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).