This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [AArch64_be] Fix vtbl[34] and vtbx4
- From: James Greenhalgh <james dot greenhalgh at arm dot com>
- To: Christophe Lyon <christophe dot lyon at linaro dot org>
- Cc: "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>
- Date: Tue, 13 Oct 2015 14:07:53 +0100
- Subject: Re: [AArch64_be] Fix vtbl[34] and vtbx4
- Authentication-results: sourceware.org; auth=none
- References: <CAKdteObt_dP63aqn3eH6mHiK5zXP+Y_rL+DfN55D=WfK_4cVGw at mail dot gmail dot com> <20151007150941 dot GA31205 at arm dot com> <CAKdteOYBU7y-z0J5d9ijU+O=DZPkLTPjjiRyhD8ywHoa4K5QPw at mail dot gmail dot com> <20151008091230 dot GA13098 at arm dot com> <CAKdteOawhToG=aw7sYYvHva4EiW46a2EDWiA9hW8GtbAqmNRkQ at mail dot gmail dot com> <20151012132854 dot GA32787 at arm dot com> <CAKdteOYuOzJgQdn29QJrnUParfk2_bvZDUepivkfzUteG5eHBA at mail dot gmail dot com>
On Tue, Oct 13, 2015 at 02:05:01PM +0100, Christophe Lyon wrote:
> I commited this as r228716, and noticed later that
> gcc.target/aarch64/table-intrinsics.c failed because of this patch.
>
> This is because that testcase scans the assembly for 'tbl v' or 'tbx
> v', but since I replaced some asm statements,
> the space is now a tab.
>
> I plan to commit this (probably obvious?):
> 2015-10-13 Christophe Lyon <christophe.lyon@linaro.org>
>
> * gcc/testsuite/gcc.target/aarch64/table-intrinsics.c: Fix regexp
> after r228716 (Fix vtbl[34] and vtbx4).
Bad luck. This is fine (and yes, obvious).
Thanks,
James
> Index: gcc/testsuite/gcc.target/aarch64/table-intrinsics.c
> ===================================================================
> --- gcc/testsuite/gcc.target/aarch64/table-intrinsics.c (revision 228759)
> +++ gcc/testsuite/gcc.target/aarch64/table-intrinsics.c (working copy)
> @@ -435,5 +435,5 @@
> return vqtbx4q_p8 (r, tab, idx);
> }
>
> -/* { dg-final { scan-assembler-times "tbl v" 42} } */
> -/* { dg-final { scan-assembler-times "tbx v" 30} } */
> +/* { dg-final { scan-assembler-times "tbl\[ |\t\]*v" 42} } */
> +/* { dg-final { scan-assembler-times "tbx\[ |\t\]*v" 30} } */