File s390-add-suffix-to-conditional-branch-instruction-de.patch of Package gdb.38112
From 2c5de93b010ce39182b1feb08ab15a7ba8904794 Mon Sep 17 00:00:00 2001
From: Jens Remus <jremus@linux.ibm.com>
Date: Thu, 13 Mar 2025 15:09:48 +0100
Subject: [PATCH 11/28] s390: Add suffix to conditional branch instruction
descriptions
Suffix the instruction description of conditional branch extended
mnemonics with their condition (e.g. "on A high"). This complements
the optional printing of instruction descriptions as comments in the
disassembly.
Due to the added text the maximum description length is increased from
80 to 128 characters (including the trailing '\0' character).
opcodes/
* s390-mkopc.c: Add suffix to conditional branch extended
mnemonic instruction descriptions.
gas/
* testsuite/gas/s390/zarch-insndesc.s: Add test cases for
printing of suffixed instruction description of conditional
branch extended mnemonics.
* testsuite/gas/s390/zarch-insndesc.d: Likewise.
Signed-off-by: Jens Remus <jremus@linux.ibm.com>
Reviewed-by: Andreas Krebbel <krebbel@linux.ibm.com>
(cherry picked from commit 1c354ebcbac)
(cherry pick dropped: gas/testsuite/gas/s390/zarch-insndesc.d)
(cherry pick dropped: gas/testsuite/gas/s390/zarch-insndesc.s)
---
opcodes/s390-mkopc.c | 78 +++++++++++++++++++++++++-------------------
1 file changed, 44 insertions(+), 34 deletions(-)
diff --git a/opcodes/s390-mkopc.c b/opcodes/s390-mkopc.c
index c6930a3e9b5..025dbdb037f 100644
--- a/opcodes/s390-mkopc.c
+++ b/opcodes/s390-mkopc.c
@@ -32,7 +32,7 @@
#define MAX_OPCODE_LEN 15
#define MAX_MNEMONIC_LEN 15
#define MAX_FORMAT_LEN 15
-#define MAX_DESCRIPTION_LEN 79
+#define MAX_DESCRIPTION_LEN 127
#define MAX_CPU_LEN 15
#define MAX_MODES_LEN 15
@@ -142,50 +142,52 @@ struct s390_cond_ext_format
{
char nibble;
char extension[4];
+ char *description_suffix;
+
};
/* The mnemonic extensions for conditional jumps used to replace
the '*' tag. */
#define NUM_COND_EXTENSIONS 20
const struct s390_cond_ext_format s390_cond_extensions[NUM_COND_EXTENSIONS] =
-{ { '1', "o" }, /* jump on overflow / if ones */
- { '2', "h" }, /* jump on A high */
- { '2', "p" }, /* jump on plus */
- { '3', "nle" }, /* jump on not low or equal */
- { '4', "l" }, /* jump on A low */
- { '4', "m" }, /* jump on minus / if mixed */
- { '5', "nhe" }, /* jump on not high or equal */
- { '6', "lh" }, /* jump on low or high */
- { '7', "ne" }, /* jump on A not equal B */
- { '7', "nz" }, /* jump on not zero / if not zeros */
- { '8', "e" }, /* jump on A equal B */
- { '8', "z" }, /* jump on zero / if zeros */
- { '9', "nlh" }, /* jump on not low or high */
- { 'a', "he" }, /* jump on high or equal */
- { 'b', "nl" }, /* jump on A not low */
- { 'b', "nm" }, /* jump on not minus / if not mixed */
- { 'c', "le" }, /* jump on low or equal */
- { 'd', "nh" }, /* jump on A not high */
- { 'd', "np" }, /* jump on not plus */
- { 'e', "no" }, /* jump on not overflow / if not ones */
+{ { '1', "o", "on overflow / if ones"}, /* jump on overflow / if ones */
+ { '2', "h", "on A high"}, /* jump on A high */
+ { '2', "p", "on plus" }, /* jump on plus */
+ { '3', "nle", "on not low or equal" }, /* jump on not low or equal */
+ { '4', "l", "on A low" }, /* jump on A low */
+ { '4', "m", "on minus / if mixed" }, /* jump on minus / if mixed */
+ { '5', "nhe", "on not high or equal" }, /* jump on not high or equal */
+ { '6', "lh", "on low or high" }, /* jump on low or high */
+ { '7', "ne", "on A not equal B" }, /* jump on A not equal B */
+ { '7', "nz", "on not zero / if not zeros" }, /* jump on not zero / if not zeros */
+ { '8', "e", "on A equal B" }, /* jump on A equal B */
+ { '8', "z", "on zero / if zeros" }, /* jump on zero / if zeros */
+ { '9', "nlh", "on not low or high " }, /* jump on not low or high */
+ { 'a', "he", "on high or equal" }, /* jump on high or equal */
+ { 'b', "nl", "on A not low" }, /* jump on A not low */
+ { 'b', "nm", "on not minus / if not mixed" }, /* jump on not minus / if not mixed */
+ { 'c', "le", "on low or equal" }, /* jump on low or equal */
+ { 'd', "nh", "on A not high" }, /* jump on A not high */
+ { 'd', "np", "on not plus" }, /* jump on not plus */
+ { 'e', "no", "on not overflow / if not ones" },/* jump on not overflow / if not ones */
};
/* The mnemonic extensions for conditional branches used to replace
the '$' tag. */
#define NUM_CRB_EXTENSIONS 12
const struct s390_cond_ext_format s390_crb_extensions[NUM_CRB_EXTENSIONS] =
-{ { '2', "h" }, /* jump on A high */
- { '2', "nle" }, /* jump on not low or equal */
- { '4', "l" }, /* jump on A low */
- { '4', "nhe" }, /* jump on not high or equal */
- { '6', "ne" }, /* jump on A not equal B */
- { '6', "lh" }, /* jump on low or high */
- { '8', "e" }, /* jump on A equal B */
- { '8', "nlh" }, /* jump on not low or high */
- { 'a', "nl" }, /* jump on A not low */
- { 'a', "he" }, /* jump on high or equal */
- { 'c', "nh" }, /* jump on A not high */
- { 'c', "le" }, /* jump on low or equal */
+{ { '2', "h", "on A high" }, /* jump on A high */
+ { '2', "nle", "on not low or equal" }, /* jump on not low or equal */
+ { '4', "l", "on A low" }, /* jump on A low */
+ { '4', "nhe", "on not high or equal" }, /* jump on not high or equal */
+ { '6', "ne", "on A not equal B" }, /* jump on A not equal B */
+ { '6', "lh", "on low or high" }, /* jump on low or high */
+ { '8', "e", "on A equal B" }, /* jump on A equal B */
+ { '8', "nlh", "on not low or high" }, /* jump on not low or high */
+ { 'a', "nl", "on A not low" }, /* jump on A not low */
+ { 'a', "he", "on high or equal" }, /* jump on high or equal */
+ { 'c', "nh", "on A not high" }, /* jump on A not high */
+ { 'c', "le", "on low or equal" }, /* jump on low or equal */
};
/* As with insertOpcode instructions are added to the sorted opcode
@@ -283,6 +285,7 @@ insertExpandedMnemonic (char *opcode, char *mnemonic, char *format,
for (i = 0; i < ext_table_length; i++)
{
char new_mnemonic[MAX_MNEMONIC_LEN + 1];
+ char new_description[MAX_DESCRIPTION_LEN + 1];
opcode[mask_start] = ext_table[i].nibble;
@@ -293,7 +296,14 @@ insertExpandedMnemonic (char *opcode, char *mnemonic, char *format,
return;
}
- insertOpcode (opcode, new_mnemonic, format, min_cpu, mode_bits, flags, description);
+ if (snprintf (new_description, sizeof (new_description), "%s %s", description,
+ ext_table[i].description_suffix) >= sizeof (new_description))
+ {
+ print_error ("Mnemonic \"%s\": Concatenated description exceeds max. length\n", mnemonic);
+ return;
+ }
+
+ insertOpcode (opcode, new_mnemonic, format, min_cpu, mode_bits, flags, new_description);
}
return;
--
2.43.0
