[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] tests/tcg/s390x: Use a different PCRel32 notation in branch-
From: |
Ilya Leoshkevich |
Subject: |
Re: [PATCH] tests/tcg/s390x: Use a different PCRel32 notation in branch-relative-long.c |
Date: |
Wed, 04 May 2022 00:46:30 +0200 |
User-agent: |
Evolution 3.42.4 (3.42.4-1.fc35) |
On Tue, 2022-05-03 at 21:26 +0200, Thomas Huth wrote:
> On 03/05/2022 11.02, Thomas Huth wrote:
> > On 02/05/2022 18.48, Ilya Leoshkevich wrote:
> > > Binutils >=2.37 and Clang do not accept (. - 0x100000000) PCRel32
> > > constants. While this looks like a bug that needs fixing, use a
> > > different notation (-0x100000000) as a workaround.
> > >
> > > Reported-by: Thomas Huth <thuth@redhat.com>
> > > Signed-off-by: Ilya Leoshkevich <iii@linux.ibm.com>
> > > ---
> > > tests/tcg/s390x/branch-relative-long.c | 4 ++--
> > > 1 file changed, 2 insertions(+), 2 deletions(-)
> > >
> > > diff --git a/tests/tcg/s390x/branch-relative-long.c
> > > b/tests/tcg/s390x/branch-relative-long.c
> > > index 94219afcad..8ce9f1c2e5 100644
> > > --- a/tests/tcg/s390x/branch-relative-long.c
> > > +++ b/tests/tcg/s390x/branch-relative-long.c
> > > @@ -13,8 +13,8 @@
> > > #_name "_end:\n");
> > > DEFINE_ASM(br_r14, "br %r14");
> > > -DEFINE_ASM(brasl_r0, "brasl %r0,.-0x100000000");
> > > -DEFINE_ASM(brcl_0xf, "brcl 0xf,.-0x100000000");
> > > +DEFINE_ASM(brasl_r0, "brasl %r0,-0x100000000");
> > > +DEFINE_ASM(brcl_0xf, "brcl 0xf,-0x100000000");
> >
> > Works for me, thanks!
>
> Sorry, I spoke too soon - it compiles fine, and also runs fine when I
> run it
> natively, but when I run it through "qemu-s390x", it crashes... does
> that
> work for you?
Hi, yes, I just double-checked - it works fine for me.
Could you please share the resulting test binary?
- [PATCH] tests/tcg/s390x: Use a different PCRel32 notation in branch-relative-long.c, Ilya Leoshkevich, 2022/05/02
- Re: [PATCH] tests/tcg/s390x: Use a different PCRel32 notation in branch-relative-long.c, Thomas Huth, 2022/05/03
- Re: [PATCH] tests/tcg/s390x: Use a different PCRel32 notation in branch-relative-long.c, Thomas Huth, 2022/05/03
- Re: [PATCH] tests/tcg/s390x: Use a different PCRel32 notation in branch-relative-long.c,
Ilya Leoshkevich <=
- Re: [PATCH] tests/tcg/s390x: Use a different PCRel32 notation in branch-relative-long.c, Thomas Huth, 2022/05/04
- Re: [PATCH] tests/tcg/s390x: Use a different PCRel32 notation in branch-relative-long.c, Ilya Leoshkevich, 2022/05/04
- branch-relative-long fails on s390x host (was: [PATCH] tests/tcg/s390x: Use a different PCRel32 notation in branch-relative-long.c), Thomas Huth, 2022/05/04
- Re: branch-relative-long fails on s390x host (was: [PATCH] tests/tcg/s390x: Use a different PCRel32 notation in branch-relative-long.c), Ilya Leoshkevich, 2022/05/04
- Re: branch-relative-long fails on s390x host (was: [PATCH] tests/tcg/s390x: Use a different PCRel32 notation in branch-relative-long.c), Thomas Huth, 2022/05/04
- Re: branch-relative-long fails on s390x host (was: [PATCH] tests/tcg/s390x: Use a different PCRel32 notation in branch-relative-long.c), Ilya Leoshkevich, 2022/05/04
- Re: branch-relative-long fails on s390x host (was: [PATCH] tests/tcg/s390x: Use a different PCRel32 notation in branch-relative-long.c), Ilya Leoshkevich, 2022/05/04