[PATCH] D56859: [SelectionDAG] Add static getMaxNumOperands function to SDNode.

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 17 09:02:46 PST 2019


fhahn marked an inline comment as done.
fhahn added inline comments.


================
Comment at: llvm/lib/CodeGen/SelectionDAG/SelectionDAG.cpp:9269
   assert(!Node->OperandList && "Node already has operands");
-  assert(std::numeric_limits<decltype(SDNode::NumOperands)>::max() >=
-             Vals.size() &&
+  assert(SDNode::getNumOperands() >= Vals.size() &&
          "too many operands to fit into SDNode");
----------------
typo sneaked in, 'll fix that.


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

https://reviews.llvm.org/D56859





More information about the llvm-commits mailing list