Re: [PATCH 2/2] target/arm: Relax r13 restriction for ldrex/strex for v8.0

2019-11-18 Thread Peter Maydell
On Mon, 18 Nov 2019 at 20:02, Richard Henderson wrote: > > On 11/18/19 6:53 PM, Peter Maydell wrote: > > On Mon, 18 Nov 2019 at 13:16, Richard Henderson > > wrote: > >> > >> On 11/18/19 2:10 PM, Peter Maydell wrote: > /* We UNDEF for these UNPREDICTABLE cases. */ > if (a->rn

Re: [PATCH 2/2] target/arm: Relax r13 restriction for ldrex/strex for v8.0

2019-11-18 Thread Richard Henderson
On 11/18/19 6:53 PM, Peter Maydell wrote: > On Mon, 18 Nov 2019 at 13:16, Richard Henderson > wrote: >> >> On 11/18/19 2:10 PM, Peter Maydell wrote: /* We UNDEF for these UNPREDICTABLE cases. */ if (a->rn == 15 || a->rt == 15 -|| (s->thumb && a->rt == 13) +

Re: [PATCH 2/2] target/arm: Relax r13 restriction for ldrex/strex for v8.0

2019-11-18 Thread Peter Maydell
On Mon, 18 Nov 2019 at 13:16, Richard Henderson wrote: > > On 11/18/19 2:10 PM, Peter Maydell wrote: > >> /* We UNDEF for these UNPREDICTABLE cases. */ > >> if (a->rn == 15 || a->rt == 15 > >> -|| (s->thumb && a->rt == 13) > >> +|| (!ENABLE_ARCH_8 && s->thumb && a->rt ==

Re: [PATCH 2/2] target/arm: Relax r13 restriction for ldrex/strex for v8.0

2019-11-18 Thread Richard Henderson
On 11/18/19 2:10 PM, Peter Maydell wrote: >> /* We UNDEF for these UNPREDICTABLE cases. */ >> if (a->rn == 15 || a->rt == 15 >> -|| (s->thumb && a->rt == 13) >> +|| (!ENABLE_ARCH_8 && s->thumb && a->rt == 13) >> || (mop == MO_64 >> && (a->rt2 == 15 |

Re: [PATCH 2/2] target/arm: Relax r13 restriction for ldrex/strex for v8.0

2019-11-18 Thread Peter Maydell
On Sun, 17 Nov 2019 at 09:06, Richard Henderson wrote: > > Armv8-A removes UNPREDICTABLE for R13 for these cases. > > Signed-off-by: Richard Henderson > --- > target/arm/translate.c | 8 > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/target/arm/translate.c b/target/a

[PATCH 2/2] target/arm: Relax r13 restriction for ldrex/strex for v8.0

2019-11-17 Thread Richard Henderson
Armv8-A removes UNPREDICTABLE for R13 for these cases. Signed-off-by: Richard Henderson --- target/arm/translate.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/target/arm/translate.c b/target/arm/translate.c index b285b23858..3db8103966 100644 --- a/target/arm/tran