[PATCH] D47999: cl::opt ForceTrackRegLiveness for forcing liveness tracking on MIR (in MIR-Parser)

Matthias Braun via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 18 13:49:33 PDT 2018


MatzeB added a comment.

IMO `MachineFunction::init()` should not set TracksLiveness by default and instead VirtRegMap / FastRegAlloc should set it once it actually filled in the basic block live-in lists.

Could you check what happens when you do that? (I suspect you will be hitting an assert() somewhere or you wouldn't have bothered with this patch in the first place... Hopefully we can just fix that assert).


Repository:
  rL LLVM

https://reviews.llvm.org/D47999





More information about the llvm-commits mailing list