diff options
author | Manman Ren <mren@apple.com> | 2013-06-28 05:43:10 +0000 |
---|---|---|
committer | Manman Ren <mren@apple.com> | 2013-06-28 05:43:10 +0000 |
commit | 983a16c08adbeb89c34e894647d4ae08a0417481 (patch) | |
tree | 064443222a08013d92f082ec72a34db4e400894c /llvm/lib/CodeGen/MachineInstr.cpp | |
parent | 0a00836bb7be52daa291c681d6041300b627bc21 (diff) | |
download | bcm5719-llvm-983a16c08adbeb89c34e894647d4ae08a0417481.tar.gz bcm5719-llvm-983a16c08adbeb89c34e894647d4ae08a0417481.zip |
Debug Info: clean up usage of Verify.
No functionality change.
It should suffice to check the type of a debug info metadata, instead of
calling Verify. For cases where we know the type of a DI metadata, use
assert.
Also update testing cases to make them conform to the format of DI classes.
llvm-svn: 185135
Diffstat (limited to 'llvm/lib/CodeGen/MachineInstr.cpp')
-rw-r--r-- | llvm/lib/CodeGen/MachineInstr.cpp | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/llvm/lib/CodeGen/MachineInstr.cpp b/llvm/lib/CodeGen/MachineInstr.cpp index 32d066894b5..368c7ea0790 100644 --- a/llvm/lib/CodeGen/MachineInstr.cpp +++ b/llvm/lib/CodeGen/MachineInstr.cpp @@ -1411,8 +1411,10 @@ static void printDebugLoc(DebugLoc DL, const MachineFunction *MF, const LLVMContext &Ctx = MF->getFunction()->getContext(); if (!DL.isUnknown()) { // Print source line info. DIScope Scope(DL.getScope(Ctx)); + assert((!Scope || Scope.isScope()) && + "Scope of a DebugLoc should be null or a DIScope."); // Omit the directory, because it's likely to be long and uninteresting. - if (Scope.Verify()) + if (Scope) CommentOS << Scope.getFilename(); else CommentOS << "<unknown>"; |