[PATCH] D55073: [SelectionDAG] Split very large token factors for loads into 64k chunks
Eli Friedman via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Dec 4 15:54:40 PST 2018
efriedma accepted this revision.
efriedma added a comment.
This revision is now accepted and ready to land.
LGTM
================
Comment at: lib/CodeGen/SelectionDAG/SelectionDAG.cpp:9027
+ assert(1 << (sizeof(SDNode::NumOperands) * 8) > Vals.size() &&
+ "too many operands to fit into SDNode");
SDUse *Ops = OperandRecycler.allocate(
----------------
Maybe this would be more readable using std::numeric_limits? Your choice.
Repository:
rL LLVM
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D55073/new/
https://reviews.llvm.org/D55073
More information about the llvm-commits
mailing list