diff options
author | Jim Grosbach <grosbach@apple.com> | 2009-11-20 02:32:06 +0000 |
---|---|---|
committer | Jim Grosbach <grosbach@apple.com> | 2009-11-20 02:32:06 +0000 |
commit | fc81352e3fb6bb4c2a7640be53eb74c6ac909ae0 (patch) | |
tree | 973acc946b527e5696fd54e470e502605fb7206d /llvm/lib | |
parent | d8d26d06a593581219be709f410e22ed85c9ade8 (diff) | |
download | bcm5719-llvm-fc81352e3fb6bb4c2a7640be53eb74c6ac909ae0.tar.gz bcm5719-llvm-fc81352e3fb6bb4c2a7640be53eb74c6ac909ae0.zip |
Remove verifySizes() since it's not adding much value.
llvm-svn: 89443
Diffstat (limited to 'llvm/lib')
-rw-r--r-- | llvm/lib/Target/ARM/ARMConstantIslandPass.cpp | 36 |
1 files changed, 0 insertions, 36 deletions
diff --git a/llvm/lib/Target/ARM/ARMConstantIslandPass.cpp b/llvm/lib/Target/ARM/ARMConstantIslandPass.cpp index d11fafd4cb3..1832909a29b 100644 --- a/llvm/lib/Target/ARM/ARMConstantIslandPass.cpp +++ b/llvm/lib/Target/ARM/ARMConstantIslandPass.cpp @@ -221,45 +221,10 @@ namespace { unsigned GetOffsetOf(MachineInstr *MI) const; void dumpBBs(); void verify(MachineFunction &MF); - void verifySizes(MachineFunction &MF); }; char ARMConstantIslands::ID = 0; } -// verifySizes - Recalculate BB sizes from scratch and validate that the result -// matches the values we've been using. -void ARMConstantIslands::verifySizes(MachineFunction &MF) { - unsigned Offset = 0; - for (MachineFunction::iterator MBBI = MF.begin(), E = MF.end(); - MBBI != E; ++MBBI) { - MachineBasicBlock &MBB = *MBBI; - unsigned MBBSize = 0; - for (MachineBasicBlock::iterator I = MBB.begin(), E = MBB.end(); - I != E; ++I) { - // Add instruction size to MBBSize. - MBBSize += TII->GetInstSizeInBytes(I); - } - // In thumb mode, if this block is a constpool island, we may need padding - // so it's aligned on 4 byte boundary. - if (isThumb && - !MBB.empty() && - MBB.begin()->getOpcode() == ARM::CONSTPOOL_ENTRY && - ((Offset%4) != 0 || HasInlineAsm)) - MBBSize += 2; - Offset += MBBSize; - - DEBUG(errs() << "block #" << MBB.getNumber() << ": " - << MBBSize << " bytes (expecting " << BBSizes[MBB.getNumber()] - << (MBB.begin()->getOpcode() == ARM::CONSTPOOL_ENTRY ? - " CONSTANTPOOL" : "") << ")\n"); -#ifndef NDEBUG - if (MBBSize != BBSizes[MBB.getNumber()]) - MBB.dump(); -#endif - assert (MBBSize == BBSizes[MBB.getNumber()] && "block size mismatch!"); - } -} - /// verify - check BBOffsets, BBSizes, alignment of islands void ARMConstantIslands::verify(MachineFunction &MF) { assert(BBOffsets.size() == BBSizes.size()); @@ -392,7 +357,6 @@ bool ARMConstantIslands::runOnMachineFunction(MachineFunction &MF) { // After a while, this might be made debug-only, but it is not expensive. verify(MF); - verifySizes(MF); // If LR has been forced spilled and no far jumps (i.e. BL) has been issued. // Undo the spill / restore of LR if possible. |