[PATCH] D84624: [DWARFYAML] Rename checkListEntryOperands() to checkOperandsNumber(). NFC.

Pavel Labath via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 27 01:41:05 PDT 2020


labath accepted this revision.
labath added a comment.
This revision is now accepted and ready to land.

Seems reasonable, but I think a better spelling of the name would be `checkOperandNumber` (no `s`) -- "check number of operands" vs "check operand number"


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D84624





More information about the llvm-commits mailing list