Re: [Qemu-devel] [PATCH 11/14] target/sh4: generate fences for SH4

[Philippe Mathieu-Daudé]

On 05/01/2017 07:10 PM, Aurelien Jarno wrote:

synco is a SH4-A only instruction.

Signed-off-by: Aurelien Jarno <aurel...@aurel32.net>

Reviewed-by: Philippe Mathieu-Daudé <f4...@amsat.org>

---
 target/sh4/translate.c | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/target/sh4/translate.c b/target/sh4/translate.c
index 7de459c9a5..c226be9718 100644
--- a/target/sh4/translate.c
+++ b/target/sh4/translate.c
@@ -1570,10 +1570,11 @@ static void _decode_opc(DisasContext * ctx)
        else
            break;
     case 0x00ab:               /* synco */
-       if (ctx->features & SH_FEATURE_SH4A)
-           return;
-       else
-           break;
+        if (ctx->features & SH_FEATURE_SH4A) {
+            tcg_gen_mb(TCG_MO_ALL | TCG_BAR_SC);
+            return;
+        }
+        break;
     case 0x4024:               /* rotcl Rn */
        {
            TCGv tmp = tcg_temp_new();