[PATCH] D33088: [LiveVariables] Switch Kill/Defs sets to be `DenseSet`

Matthias Braun via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 11 10:06:36 PDT 2017


MatzeB added a comment.

Not much to say about the patch itself which looks fine.

> I wanted to try a BitVector but I'm not sure I have an upper bound on the number of elements.

An upper bound for virtual register numbers would be `MachineRegisterInfo::getNumVirtRegs()` (and `TargetRegisterInfo::getNumReg()` for physical registers though I think this set is only about vregs).


https://reviews.llvm.org/D33088





More information about the llvm-commits mailing list