* [PATCH] tcg/tci: Fix MemOpIdx operand index for 3-operand memops
@ 2023-06-08 16:29 Richard Henderson
2023-06-08 16:58 ` Philippe Mathieu-Daudé
0 siblings, 1 reply; 3+ messages in thread
From: Richard Henderson @ 2023-06-08 16:29 UTC (permalink / raw)
To: qemu-devel
Cut and paste error from the 4-operand memops.
Fixes: ab64da797740 ("tcg/tci: Adjust passing of MemOpIdx")
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
---
tcg/tci/tcg-target.c.inc | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tcg/tci/tcg-target.c.inc b/tcg/tci/tcg-target.c.inc
index 0037f904f1..253f27f174 100644
--- a/tcg/tci/tcg-target.c.inc
+++ b/tcg/tci/tcg-target.c.inc
@@ -847,7 +847,7 @@ static void tcg_out_op(TCGContext *s, TCGOpcode opc,
if (TCG_TARGET_REG_BITS == 64) {
tcg_out_op_rrm(s, opc, args[0], args[1], args[2]);
} else {
- tcg_out_movi(s, TCG_TYPE_I32, TCG_REG_TMP, args[4]);
+ tcg_out_movi(s, TCG_TYPE_I32, TCG_REG_TMP, args[3]);
tcg_out_op_rrrr(s, opc, args[0], args[1], args[2], TCG_REG_TMP);
}
break;
--
2.34.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] tcg/tci: Fix MemOpIdx operand index for 3-operand memops
2023-06-08 16:29 [PATCH] tcg/tci: Fix MemOpIdx operand index for 3-operand memops Richard Henderson
@ 2023-06-08 16:58 ` Philippe Mathieu-Daudé
2023-06-09 18:55 ` Richard Henderson
0 siblings, 1 reply; 3+ messages in thread
From: Philippe Mathieu-Daudé @ 2023-06-08 16:58 UTC (permalink / raw)
To: Richard Henderson, qemu-devel
On 8/6/23 18:29, Richard Henderson wrote:
> Cut and paste error from the 4-operand memops.
>
> Fixes: ab64da797740 ("tcg/tci: Adjust passing of MemOpIdx")
I had this patch tagged for review but you were quicker
(not sure I'd have caught the issue although)
> Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
> ---
> tcg/tci/tcg-target.c.inc | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] tcg/tci: Fix MemOpIdx operand index for 3-operand memops
2023-06-08 16:58 ` Philippe Mathieu-Daudé
@ 2023-06-09 18:55 ` Richard Henderson
0 siblings, 0 replies; 3+ messages in thread
From: Richard Henderson @ 2023-06-09 18:55 UTC (permalink / raw)
To: Philippe Mathieu-Daudé, qemu-devel
On 6/8/23 09:58, Philippe Mathieu-Daudé wrote:
> On 8/6/23 18:29, Richard Henderson wrote:
>> Cut and paste error from the 4-operand memops.
>>
>> Fixes: ab64da797740 ("tcg/tci: Adjust passing of MemOpIdx")
>
> I had this patch tagged for review but you were quicker
> (not sure I'd have caught the issue although)
>
>> Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
>> ---
>> tcg/tci/tcg-target.c.inc | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
>
>
Applied to master, though it hasn't completely greened the board.
r~
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2023-06-09 18:55 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-06-08 16:29 [PATCH] tcg/tci: Fix MemOpIdx operand index for 3-operand memops Richard Henderson
2023-06-08 16:58 ` Philippe Mathieu-Daudé
2023-06-09 18:55 ` Richard Henderson
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.