diff options
author | Alex Lorenz <arphaman@gmail.com> | 2015-07-28 16:56:45 +0000 |
---|---|---|
committer | Alex Lorenz <arphaman@gmail.com> | 2015-07-28 16:56:45 +0000 |
commit | e8ce3e616b7d7f4f1635dcdbea7ea74752244cd9 (patch) | |
tree | e6827c8c867bc01ee0b699400ee9d8f33ea5b2cc /llvm/lib | |
parent | c4ffd48aba663ea3dfd5bf60fab9affcf5b081be (diff) | |
download | bcm5719-llvm-e8ce3e616b7d7f4f1635dcdbea7ea74752244cd9.tar.gz bcm5719-llvm-e8ce3e616b7d7f4f1635dcdbea7ea74752244cd9.zip |
MIR Printer: Remove an outdated TODO comment and assertion. NFC.
This commit removes an outdated TODO comment and a corresponding assertion
which asserts that the mir printer can't the print machine basic blocks that
aren't sequentially numbered.
This comment and assertion were correct when I was working on the patch which
serialized the machine basic blocks, but then I decided to add an 'ID'
attribute to the machine basic block's YAML mapping based on the patch review.
This comment and assertion then became invalid as with the 'ID' attribute we
can serialize the non sequential machine basic blocks and their references
without any problems.
llvm-svn: 243447
Diffstat (limited to 'llvm/lib')
-rw-r--r-- | llvm/lib/CodeGen/MIRPrinter.cpp | 8 |
1 files changed, 0 insertions, 8 deletions
diff --git a/llvm/lib/CodeGen/MIRPrinter.cpp b/llvm/lib/CodeGen/MIRPrinter.cpp index cb20df8eab6..ac37e323487 100644 --- a/llvm/lib/CodeGen/MIRPrinter.cpp +++ b/llvm/lib/CodeGen/MIRPrinter.cpp @@ -166,15 +166,7 @@ void MIRPrinter::print(const MachineFunction &MF) { MST.incorporateFunction(*MF.getFunction()); if (const auto *JumpTableInfo = MF.getJumpTableInfo()) convert(MST, YamlMF.JumpTableInfo, *JumpTableInfo); - int I = 0; for (const auto &MBB : MF) { - // TODO: Allow printing of non sequentially numbered MBBs. - // This is currently needed as the basic block references get their index - // from MBB.getNumber(), thus it should be sequential so that the parser can - // map back to the correct MBBs when parsing the output. - assert(MBB.getNumber() == I++ && - "Can't print MBBs that aren't sequentially numbered"); - (void)I; yaml::MachineBasicBlock YamlMBB; convert(MST, YamlMBB, MBB); YamlMF.BasicBlocks.push_back(YamlMBB); |