|
From: | Richard Henderson |
Subject: | Re: [PATCH v2 20/55] accel/tcg: Report unaligned atomics for user-only |
Date: | Wed, 18 Aug 2021 07:47:25 -1000 |
User-agent: | Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.11.0 |
On 8/17/21 10:51 PM, Philippe Mathieu-Daudé wrote:
- void *ret = g2h(env_cpu(env), addr); + + ret = g2h(env_cpu(env), addr); set_helper_retaddr(retaddr); return ret;Can't we simply do: return g2h(env_cpu(env), addr); ?
I think the idea was to narrow the range of instructions in which helper_retaddr is set, because {set,clear}_helper_retaddr contain barriers. I didn't give it more thought this time around, just kept the ordering.
r~
[Prev in Thread] | Current Thread | [Next in Thread] |