qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH v2 20/55] accel/tcg: Report unaligned atomics for user-only


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~




reply via email to

[Prev in Thread] Current Thread [Next in Thread]