This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [AArch64, testsuite] gcc.target/aarch64/extend.c: xfails for ilp32
- From: Andrew Pinski <pinskia at gmail dot com>
- To: Charles Baylis <charles dot baylis at linaro dot org>
- Cc: James Greenhalgh <James dot Greenhalgh at arm dot com>, Richard Earnshaw <Richard dot Earnshaw at arm dot com>, Marcus Shawcroft <marcus dot shawcroft at arm dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Tue, 24 Oct 2017 11:40:17 -0700
- Subject: Re: [AArch64, testsuite] gcc.target/aarch64/extend.c: xfails for ilp32
- Authentication-results: sourceware.org; auth=none
- References: <CADnVucA9xsr07a9r3QS5k=vytWQ_WY298tFJfy857K17xVYZwQ@mail.gmail.com>
On Tue, Oct 24, 2017 at 11:27 AM, Charles Baylis
<charles.baylis@linaro.org> wrote:
> In ILP32, GCC fails to merge pointer arithmetic into the addressing
> mode of a load instruction, as
> add w0, w0, w1, lsl 2
> ldr w0, [x0]
> is not equivalent to:
> ldr w0, [x0, w1, lsl 2]
>
> Shows the expected FAIL->XFAILs on aarch64-linux-gnu_ilp32, no
> regressions on aarch64-linux-gnu.
Then this is not a xfail but rather the dg-final should be skipped for ilp32.
xfail means the failure can be fixed in the future but in this case,
the failure is not fixable.
Something like:
{ target { !ilp32 } }
or (what I used in my changes which I was going to submit but had
higher priorities than submitting testcase fixes):
{ target { lp64 } }
Thanks,
Andrew
>
> gcc/testsuite:
> <date> Charles Baylis <charles.baylis@linaro.org>
>
> * gcc.target/aarch64/extend.c (ldr_uxtw): Add xfail for ilp32.
> (ldr_uxtw0): Likewise.
> (ldr_sxtw): Likewise.
> (ldr_sxtw0): Likewise.