[PATCH] D127932: [TableGen] Add a knob for MCOperandInfo expansion in gen-instr-info

[Kan Shengchen via Phabricator via llvm-commits]

skan added inline comments.


================
Comment at: llvm/test/TableGen/get-operand-type-no-expand.td:41
+
+// RUN: llvm-tblgen -gen-instr-info -I %p/../../include %s \
+// RUN:   -instr-info-expand-mi-operand-info=0 \
----------------
Do we need to test that `-instr-info-expand-mi-operand-info=1`  while `-gen-instr-info` is not enabled?


================
Comment at: llvm/test/TableGen/get-operand-type-no-expand.td:47
+// CHECK-NOEXPAND:        /* InstA */
+// CHECK-NOEXPAND-NEXT:   i512complex, i8complex, i32imm,
+// CHECK-NOEXPAND: #endif // GET_INSTRINFO_OPERAND_TYPE
----------------
Question: i512complex is expanded to what above?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D127932