Re: [PATCH] Fix build on mips64el

[Paulo César Pereira de Andrade]

Em sex., 12 de jan. de 2024 às 21:58, Paul Cercueil
<paul@crapouillou.net> escreveu:
>
> Hi Paulo,

  Hi Paul,

> Le jeudi 05 octobre 2023 à 15:26 -0300, Paulo César Pereira de Andrade
> a écrit :
> > Em qui., 5 de out. de 2023 às 09:24, Paul Cercueil
> > <paul@crapouillou.net> escreveu:
> >
> >   Hi Paul,
> >
> > > The "cvt_offset" field may be used for anything non-ARM in
> > > jit_fallback.c, so enable this field on anything non-ARM. Worst
> > > case
> > > scenario, this field is not used and it only occupies 4 bytes of
> > > RAM.
> > >
> > > Besides, when building for mips64el with the N32 ABI, the
> > > __WORDSIZE
> > > ends up being 32 and not 64 (which may be a problem in itself),
> > > which
> > > caused some more issues as the movi_d_w() macro referenced by
> > > jit_mips.c was not defined anywhere.
> >
> >   I suppose you do not have access to any system running N32, but
> > if you have, please let me know. I only know Irix had 32 and 64
> > libraries, but I no longer have access to such a system for way
> > too long.
>
> I do not, sorry. I just wanted to fix it to stop Buildroot CI from
> sending me emails :)

  No problems.

> By the way, do you think it's time for 2.2.3?

 I came back from vacation today. I have initial work on the pre/post
increment load/store with a register argument that should be merged.
Once that is done, and at least the aarch64 port has these, that maps
directly to aarch64 instructions, a new release should be done.

> Cheers,
> -Paul

Thanks!
Paulo