diff mbox series

tcg/tci: Fix MemOpIdx operand index for 3-operand memops

Message ID 20230608162925.677598-1-richard.henderson@linaro.org
State Accepted
Commit 3673ad389622d9ef4d2743101253c642def7935a
Headers show
Series tcg/tci: Fix MemOpIdx operand index for 3-operand memops | expand

Commit Message

Richard Henderson June 8, 2023, 4:29 p.m. UTC
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(-)

Comments

Philippe Mathieu-Daudé June 8, 2023, 4:58 p.m. UTC | #1
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>
Richard Henderson June 9, 2023, 6:55 p.m. UTC | #2
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~
diff mbox series

Patch

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;