diff options
author | Adrian Prantl <aprantl@apple.com> | 2014-04-24 18:44:15 +0000 |
---|---|---|
committer | Adrian Prantl <aprantl@apple.com> | 2014-04-24 18:44:15 +0000 |
commit | 807a3c283023cd2b0767a97b5602e232b69b281b (patch) | |
tree | d439e496b18e73071236f3aa0c584433288dc35e /lib/Transforms | |
parent | 778aacb5a341fc8535025e3f86bc73630010672f (diff) | |
download | llvm-807a3c283023cd2b0767a97b5602e232b69b281b.tar.gz llvm-807a3c283023cd2b0767a97b5602e232b69b281b.tar.bz2 llvm-807a3c283023cd2b0767a97b5602e232b69b281b.tar.xz |
Debug info: Let dbg.values inserted by LowerDbgDeclare inherit the location
of the dbg.value. This gets rid of tons of redundant variable DIEs in
subscopes.
rdar://problem/14874886, rdar://problem/16679936
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@207135 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Transforms')
-rw-r--r-- | lib/Transforms/Utils/Local.cpp | 20 |
1 files changed, 3 insertions, 17 deletions
diff --git a/lib/Transforms/Utils/Local.cpp b/lib/Transforms/Utils/Local.cpp index e2ba0473fd..9199ed7912 100644 --- a/lib/Transforms/Utils/Local.cpp +++ b/lib/Transforms/Utils/Local.cpp @@ -995,14 +995,7 @@ bool llvm::ConvertDebugDeclareToDebugValue(DbgDeclareInst *DDI, DbgVal = Builder.insertDbgValueIntrinsic(ExtendedArg, 0, DIVar, SI); else DbgVal = Builder.insertDbgValueIntrinsic(SI->getOperand(0), 0, DIVar, SI); - - // Propagate any debug metadata from the store onto the dbg.value. - DebugLoc SIDL = SI->getDebugLoc(); - if (!SIDL.isUnknown()) - DbgVal->setDebugLoc(SIDL); - // Otherwise propagate debug metadata from dbg.declare. - else - DbgVal->setDebugLoc(DDI->getDebugLoc()); + DbgVal->setDebugLoc(DDI->getDebugLoc()); return true; } @@ -1022,14 +1015,7 @@ bool llvm::ConvertDebugDeclareToDebugValue(DbgDeclareInst *DDI, Instruction *DbgVal = Builder.insertDbgValueIntrinsic(LI->getOperand(0), 0, DIVar, LI); - - // Propagate any debug metadata from the store onto the dbg.value. - DebugLoc LIDL = LI->getDebugLoc(); - if (!LIDL.isUnknown()) - DbgVal->setDebugLoc(LIDL); - // Otherwise propagate debug metadata from dbg.declare. - else - DbgVal->setDebugLoc(DDI->getDebugLoc()); + DbgVal->setDebugLoc(DDI->getDebugLoc()); return true; } @@ -1068,7 +1054,7 @@ bool llvm::LowerDbgDeclare(Function &F) { auto DbgVal = DIB.insertDbgValueIntrinsic(AI, 0, DIVariable(DDI->getVariable()), I); - DbgVal->setDebugLoc(I->getDebugLoc()); + DbgVal->setDebugLoc(DDI->getDebugLoc()); } DDI->eraseFromParent(); } |