add test for invalid usage

lei137 · lei137 · commit e771bfc81cef · 2025-10-06T16:02:31.000Z
diff --git a/llvm/lib/Target/PowerPC/MCTargetDesc/PPCInstPrinter.cpp b/llvm/lib/Target/PowerPC/MCTargetDesc/PPCInstPrinter.cpp
@@ -437,8 +437,7 @@ void PPCInstPrinter::printS32ImmOperand(const MCInst *MI, unsigned OpNo,
     long long Value = MI->getOperand(OpNo).getImm();
     assert(isInt<32>(Value) && "Invalid s32imm argument!");
     O << (long long)Value;
-  }
-  else
+  } else
     printOperand(MI, OpNo, STI, O);
 }
 
diff --git a/llvm/test/MC/PowerPC/ppc64-encoding-ISA31-errors.s b/llvm/test/MC/PowerPC/ppc64-encoding-ISA31-errors.s
@@ -7,10 +7,10 @@
 paddi 1, 1, 32, 1
 
 # CHECK: error: invalid operand for instruction
-pld 1, 32(1), 1
+paddis 10, 5, 1000000000, 1
 
 # CHECK: error: invalid operand for instruction
-paddi 1, 1, 32, 1
+pld 1, 32(1), 1
 
 # CHECK: error: invalid operand for instruction
 plbz 1, 32(1), 1

Original file line number	Diff line number	Diff line change
`@@ -437,8 +437,7 @@ void PPCInstPrinter::printS32ImmOperand(const MCInst *MI, unsigned OpNo,`
`437`	`437`	`long long Value = MI->getOperand(OpNo).getImm();`
`438`	`438`	`assert(isInt<32>(Value) && "Invalid s32imm argument!");`
`439`	`439`	`O << (long long)Value;`
`440`		`- }`
`441`		`- else`
	`440`	`+ } else`
`442`	`441`	`printOperand(MI, OpNo, STI, O);`
`443`	`442`	`}`
`444`	`443`