[llvm] [PowerPC] Add error for incorrect use of memory operands (PR #114277)

Hubert Tong via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 6 20:04:12 PST 2024


================
@@ -285,6 +285,9 @@ class PPCInstrInfo : public PPCGenInstrInfo {
   bool isZExt32To64(unsigned Opcode) const {
     return get(Opcode).TSFlags & PPCII::ZExt32To64;
   }
+  bool isMemOp(unsigned Opcode) const {
+    return get(Opcode).TSFlags & PPCII::MemOp;
----------------
hubert-reinterpretcast wrote:

The issue with the "isMemOp" name is that "Op" in this context (combined with "is") means "operation"/"opcode" not "operand". I think "isMemriOp" (as you suggest) or "takesMemriOperand" would be fine.

https://github.com/llvm/llvm-project/pull/114277


More information about the llvm-commits mailing list