* [Qemu-devel] [PATCH] disas/mips: Increase 'member of ISAs' flag holder size
@ 2018-10-24 10:57 Philippe Mathieu-Daudé
2018-10-24 11:19 ` Richard Henderson
2018-10-24 17:40 ` Fredrik Noring
0 siblings, 2 replies; 3+ messages in thread
From: Philippe Mathieu-Daudé @ 2018-10-24 10:57 UTC (permalink / raw)
To: Aurelien Jarno, Aleksandar Markovic, Fredrik Noring,
Richard Henderson
Cc: Philippe Mathieu-Daudé, qemu-devel
Increase the size of 'membership' holder size to 64 bits. This is
needed for future extensions since existing bits are almost all used.
(This change is related to f9c9cd63e3),
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
---
disas/mips.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/disas/mips.c b/disas/mips.c
index 97f661a37e..d73d4094d8 100644
--- a/disas/mips.c
+++ b/disas/mips.c
@@ -301,7 +301,7 @@ struct mips_opcode
unsigned long pinfo2;
/* A collection of bits describing the instruction sets of which this
instruction or macro is a member. */
- unsigned long membership;
+ uint64_t membership;
};
/* These are the characters which may appear in the args field of an
--
2.19.1
^ permalink raw reply related [flat|nested] 3+ messages in thread* Re: [Qemu-devel] [PATCH] disas/mips: Increase 'member of ISAs' flag holder size
2018-10-24 10:57 [Qemu-devel] [PATCH] disas/mips: Increase 'member of ISAs' flag holder size Philippe Mathieu-Daudé
@ 2018-10-24 11:19 ` Richard Henderson
2018-10-24 17:40 ` Fredrik Noring
1 sibling, 0 replies; 3+ messages in thread
From: Richard Henderson @ 2018-10-24 11:19 UTC (permalink / raw)
To: Philippe Mathieu-Daudé, Aurelien Jarno, Aleksandar Markovic,
Fredrik Noring
Cc: qemu-devel
On 10/24/18 11:57 AM, Philippe Mathieu-Daudé wrote:
> Increase the size of 'membership' holder size to 64 bits. This is
> needed for future extensions since existing bits are almost all used.
> (This change is related to f9c9cd63e3),
>
> Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
> ---
> disas/mips.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
r~
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [Qemu-devel] [PATCH] disas/mips: Increase 'member of ISAs' flag holder size
2018-10-24 10:57 [Qemu-devel] [PATCH] disas/mips: Increase 'member of ISAs' flag holder size Philippe Mathieu-Daudé
2018-10-24 11:19 ` Richard Henderson
@ 2018-10-24 17:40 ` Fredrik Noring
1 sibling, 0 replies; 3+ messages in thread
From: Fredrik Noring @ 2018-10-24 17:40 UTC (permalink / raw)
To: Philippe Mathieu-Daudé
Cc: Aurelien Jarno, Aleksandar Markovic, Richard Henderson,
qemu-devel
Hi Philippe,
On Wed, Oct 24, 2018 at 12:57:32PM +0200, Philippe Mathieu-Daudé wrote:
> Increase the size of 'membership' holder size to 64 bits. This is
> needed for future extensions since existing bits are almost all used.
> (This change is related to f9c9cd63e3),
I'm queueing your patch to amend the support for the R5900, based on
Aleksandar's latest v2 tag:
https://github.com/AMarkovic/qemu tags/mips-queue-oct-2018-part-2-v2
> Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
> ---
> disas/mips.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/disas/mips.c b/disas/mips.c
> index 97f661a37e..d73d4094d8 100644
> --- a/disas/mips.c
> +++ b/disas/mips.c
> @@ -301,7 +301,7 @@ struct mips_opcode
> unsigned long pinfo2;
> /* A collection of bits describing the instruction sets of which this
> instruction or macro is a member. */
> - unsigned long membership;
> + uint64_t membership;
> };
>
> /* These are the characters which may appear in the args field of an
> --
> 2.19.1
>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2018-10-24 17:41 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-10-24 10:57 [Qemu-devel] [PATCH] disas/mips: Increase 'member of ISAs' flag holder size Philippe Mathieu-Daudé
2018-10-24 11:19 ` Richard Henderson
2018-10-24 17:40 ` Fredrik Noring
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).