qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Richard Henderson <richard.henderson@linaro.org>
To: qemu-devel@nongnu.org
Cc: peter.maydell@linaro.org, philmd@redhat.com,
	"Philippe Mathieu-Daudé" <f4bug@amsat.org>
Subject: [PATCH v2 8/8] target/arm: Use a non-overlapping group for misc control
Date: Tue,  2 Jun 2020 16:58:34 -0700	[thread overview]
Message-ID: <20200602235834.470345-9-richard.henderson@linaro.org> (raw)
In-Reply-To: <20200602235834.470345-1-richard.henderson@linaro.org>

The miscellaneous control instructions are mutually exclusive
within the t32 decode sub-group.

Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
---
 target/arm/t32.decode | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/target/arm/t32.decode b/target/arm/t32.decode
index c63082fc9c..c21a988f97 100644
--- a/target/arm/t32.decode
+++ b/target/arm/t32.decode
@@ -312,13 +312,13 @@ CLZ              1111 1010 1011 ---- 1111 .... 1000 ....      @rdm
                  &cps
 
     # Miscellaneous control
-    {
+    [
       CLREX      1111 0011 1011 1111 1000 1111 0010 1111
       DSB        1111 0011 1011 1111 1000 1111 0100 ----
       DMB        1111 0011 1011 1111 1000 1111 0101 ----
       ISB        1111 0011 1011 1111 1000 1111 0110 ----
       SB         1111 0011 1011 1111 1000 1111 0111 0000
-    }
+    ]
 
     # Note that the v7m insn overlaps both the normal and banked insn.
     {
-- 
2.25.1



      parent reply	other threads:[~2020-06-03  0:01 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-06-02 23:58 [PATCH v2 0/8] decodetree: Add non-overlapping groups Richard Henderson
2020-06-02 23:58 ` [PATCH v2 1/8] decodetree: Tidy error_with_file Richard Henderson
2020-06-02 23:58 ` [PATCH v2 2/8] decodetree: Rename MultiPattern to IncMultiPattern Richard Henderson
2020-06-02 23:58 ` [PATCH v2 3/8] decodetree: Split out MultiPattern from IncMultiPattern Richard Henderson
2020-06-02 23:58 ` [PATCH v2 4/8] decodetree: Allow group covering the entire insn space Richard Henderson
2020-06-02 23:58 ` [PATCH v2 5/8] decodetree: Move semantic propagation into classes Richard Henderson
2020-06-02 23:58 ` [PATCH v2 6/8] decodetree: Implement non-overlapping groups Richard Henderson
2020-06-08 10:17   ` Peter Maydell
2020-06-02 23:58 ` [PATCH v2 7/8] tests/decode: Test " Richard Henderson
2020-06-08 10:17   ` Peter Maydell
2020-06-02 23:58 ` Richard Henderson [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20200602235834.470345-9-richard.henderson@linaro.org \
    --to=richard.henderson@linaro.org \
    --cc=f4bug@amsat.org \
    --cc=peter.maydell@linaro.org \
    --cc=philmd@redhat.com \
    --cc=qemu-devel@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).