[PATCH] D134073: [TableGen] Add useDeprecatedPositionallyEncodedOperands option.

James Y Knight via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 20 05:34:23 PDT 2022


jyknight added inline comments.


================
Comment at: llvm/test/TableGen/InsufficientPositionalOperands.td:27
 
-// CHECK: Too few operands in record foo (no match for variable rs)
-  let OutOperandList = (outs Regs:$xd);
+// CHECK: No operand named rs in record foo
+  let OutOperandList = (outs Regs:$rd);
----------------
MaskRay wrote:
> Add `CHECK-NEXT: note: Dumping record for previous error:` then use `--implicit-check-not=error:`
> 
> Q: by changing `$xd`, do we lose test coverage by not testing an undefined operand name?
Done, and good point about losing test coverage. Switched this test to useDeprecatedPositionalOperands to preserve testing the positional error, and another test added for when positional matching is off.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D134073/new/

https://reviews.llvm.org/D134073



More information about the llvm-commits mailing list