Re: [Qemu-devel] [PATCH v1 2/4] target-arm: A64: Handle blr lr

2014-05-01 Thread Alex Bennée
Edgar E. Iglesias writes: > On Thu, May 01, 2014 at 10:31:06AM +0100, Peter Maydell wrote: >> On 1 May 2014 10:02, Alex Bennée wrote: >> > >> > Edgar E. Iglesias writes: >> > >> >> From: "Edgar E. Iglesias" >> >> >> >> For linked branches, updates to the link register happen >> >> conceptuall

Re: [Qemu-devel] [PATCH v1 2/4] target-arm: A64: Handle blr lr

2014-05-01 Thread Edgar E. Iglesias
On Thu, May 01, 2014 at 10:31:06AM +0100, Peter Maydell wrote: > On 1 May 2014 10:02, Alex Bennée wrote: > > > > Edgar E. Iglesias writes: > > > >> From: "Edgar E. Iglesias" > >> > >> For linked branches, updates to the link register happen > >> conceptually after the read of the branch target r

Re: [Qemu-devel] [PATCH v1 2/4] target-arm: A64: Handle blr lr

2014-05-01 Thread Peter Maydell
On 1 May 2014 10:02, Alex Bennée wrote: > > Edgar E. Iglesias writes: > >> From: "Edgar E. Iglesias" >> >> For linked branches, updates to the link register happen >> conceptually after the read of the branch target register. >> >> Signed-off-by: Edgar E. Iglesias > > I'm trying to think of a c

Re: [Qemu-devel] [PATCH v1 2/4] target-arm: A64: Handle blr lr

2014-05-01 Thread Alex Bennée
Edgar E. Iglesias writes: > From: "Edgar E. Iglesias" > > For linked branches, updates to the link register happen > conceptually after the read of the branch target register. > > Signed-off-by: Edgar E. Iglesias I'm trying to think of a case where this could actually cause a problem but I ca

[Qemu-devel] [PATCH v1 2/4] target-arm: A64: Handle blr lr

2014-04-30 Thread Edgar E. Iglesias
From: "Edgar E. Iglesias" For linked branches, updates to the link register happen conceptually after the read of the branch target register. Signed-off-by: Edgar E. Iglesias --- target-arm/translate-a64.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/target-arm/transla