On Tue, Aug 1, 2017 at 10:44 PM, Segher Boessenkool
[off-list ref] wrote:

Hi!

On Mon, Jul 31, 2017 at 10:58:22AM +1000, Matt Brown wrote:

quoted

@@ -1049,6 +1065,10 @@ int analyse_instr(struct instruction_op *op, struct pt_regs *regs,
                      do_cmp_unsigned(regs, val, val2, rd >> 2);
                      goto instr_done;

+             case 508: /* cmpb */
+                     do_cmpb(regs, regs->gpr[rd], regs->gpr[rb], ra);
+                     goto instr_done;

Should this then be under an ifdef for 64-bit?

I don't think so, the cmpb instruction should be 32 and 64-bit.
It isn't listed under the '64-bit Fixed-point Logical Instructions'
section in the ISA either.

Thanks,
Matt

Segher