This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [Patch] Gate vect-mask-store-move-1.c correctly, and actually output the dump

From: Yuri Rumyantsev <ysrumyan at gmail dot com>
To: James Greenhalgh <james dot greenhalgh at arm dot com>
Cc: gcc-patches <gcc-patches at gcc dot gnu dot org>, nd at arm dot com, Richard Biener <richard dot guenther at gmail dot com>, Uros Bizjak <ubizjak at gmail dot com>, ÐÐÑÑ ÐÐÐÐÐÐÑ <enkovich dot gnu at gmail dot com>, Igor Zamyatin <izamyatin at gmail dot com>
Date: Mon, 8 Feb 2016 16:29:31 +0300
Subject: Re: [Patch] Gate vect-mask-store-move-1.c correctly, and actually output the dump
Authentication-results: sourceware.org; auth=none
References: <CAEoMCqQc_kbYCHzd6241EPP7oZExA7MYKPA5Y-DaVnMYJL3LPA at mail dot gmail dot com> <1454936845-18517-1-git-send-email-james dot greenhalgh at arm dot com>

Hi James,

Thanks for reporting this issue.
I prepared slightly different patch since we don't need to add
tree-vect dump option - it is on by default for all tests in /vect
directory.

gcc/testsuite/ChangeLog:

* gcc.dg/vect/vect-mask-store-move-1.c: Gate dump with x86 target.

2016-02-08 16:07 GMT+03:00 James Greenhalgh <james.greenhalgh@arm.com>:
>
> Hi,
>
> As far as I can tell, this testcase will only vectorize for x86_64/i?86
> targets, so it should be gated to only check for vectorization on those.
>
> Additionally, this test wants to scan the vectorizer dumps, so we ought
> to add -fdump-tree-vect-all to the options.
>
> Checked on aarch64 (cross/native) and x86 with no issues.
>
> OK?
>
> Thanks,
> James
>
> ---
> 2016-02-08  James Greenhalgh  <james.greenhalgh@arm.com>
>
>         * gcc.dg/vect/vect-mask-store-move-1.c: Add dump option, and gate
>         check on x86_64/i?86.
>

Attachment: patch-test
Description: Binary data

Follow-Ups:
- Re: [Patch] Gate vect-mask-store-move-1.c correctly, and actually output the dump
  - From: James Greenhalgh

References:
- [Patch] Gate vect-mask-store-move-1.c correctly, and actually output the dump
  - From: James Greenhalgh

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]