[PATCH] D39034: [WIP][GlobalISel][TableGen] Optimize MatchTable for faster instruction selection
Daniel Sanders via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Dec 11 14:58:21 PST 2017
dsanders added inline comments.
================
Comment at: utils/TableGen/GlobalISelEmitter.cpp:811
- for (const auto &Predicate : predicates())
+ unsigned OpIdx = (*predicates_begin())->getOpIdx();
+ (void)OpIdx;
----------------
qcolombet wrote:
> dsanders wrote:
> > This will crash if predicates_begin() == predicates_end()
> We check this case in the previous statement, right?
Scratch this one. We check it a little earlier, I just didn’t notice because it was off screen
Repository:
rL LLVM
https://reviews.llvm.org/D39034
More information about the llvm-commits
mailing list