Re: [PATCH] target/ppc: Set ctx->opcode for decode_insn32()

2024-07-31 Thread Richard Henderson
On 7/31/24 20:07, Ilya Leoshkevich wrote: divdu (without a dot) sometimes updates cr0, even though it shouldn't. The reason is that gen_op_arith_divd() checks Rc(ctx->opcode), which is not initialized. This field is initialized only for instructions that go through decode_legacy(), and not decode

Re: [PATCH] target/ppc: Set ctx->opcode for decode_insn32()

2024-07-31 Thread Philippe Mathieu-Daudé
On 31/7/24 12:07, Ilya Leoshkevich wrote: divdu (without a dot) sometimes updates cr0, even though it shouldn't. The reason is that gen_op_arith_divd() checks Rc(ctx->opcode), which is not initialized. This field is initialized only for instructions that go through decode_legacy(), and not decode

[PATCH] target/ppc: Set ctx->opcode for decode_insn32()

2024-07-31 Thread Ilya Leoshkevich
divdu (without a dot) sometimes updates cr0, even though it shouldn't. The reason is that gen_op_arith_divd() checks Rc(ctx->opcode), which is not initialized. This field is initialized only for instructions that go through decode_legacy(), and not decodetree. There already was a similar issue fix