Hi Luc,

On 10/7/26 09:23, Luc Michel wrote:
Since 36a9529e60e0 the TCGv returned by compute_ldst_addr_type{a,b} for
the address can be a target register or a constant. However
do_{load,store} might modify it in the case of a reversed load/store.

Fix this by factoring out the reversed access logic and copying the
address in a new TCGv_i32 if it needs to be modified.

Fixes: 36a9529e60e0 (target/microblaze: Simplify compute_ldst_addr_type{a,b})
Signed-off-by: Luc Michel <[email protected]>
---
  target/microblaze/translate.c | 56 ++++++++++++++++++++---------------
  1 file changed, 32 insertions(+), 24 deletions(-)

The tests in our CI didn't catch this problem.
Do you have an image I can use to test your fix?

Reply via email to