diff options
author | Duncan P. N. Exon Smith <dexonsmith@apple.com> | 2015-04-15 21:18:07 +0000 |
---|---|---|
committer | Duncan P. N. Exon Smith <dexonsmith@apple.com> | 2015-04-15 21:18:07 +0000 |
commit | cd1aecfe369a642ad1e73fb6d98bb2582e3f87ef (patch) | |
tree | c02f5ef32e7c06f037b8b9c4e40a2da6756f2f8f /llvm/lib/CodeGen/LiveDebugVariables.cpp | |
parent | 48b3503c16b35a6ad272162976724b6a8ba9fa9d (diff) | |
download | llvm-cd1aecfe369a642ad1e73fb6d98bb2582e3f87ef.zip llvm-cd1aecfe369a642ad1e73fb6d98bb2582e3f87ef.tar.gz llvm-cd1aecfe369a642ad1e73fb6d98bb2582e3f87ef.tar.bz2 |
DebugInfo: Require a DebugLoc in DIBuilder::insertDeclare()
Change `DIBuilder::insertDeclare()` and `insertDbgValueIntrinsic()` to
take an `MDLocation*`/`DebugLoc` parameter which it attaches to the
created intrinsic. Assert at creation time that the `scope:` field's
subprogram matches the variable's. There's a matching `clang` commit to
use the API.
The context for this is PR22778, which is removing the `inlinedAt:`
field from `MDLocalVariable`, instead deferring to the `!dbg` location
attached to the debug info intrinsic. The best way to ensure we always
have a `!dbg` attachment is to require one at creation time. I'll be
adding verifier checks next, but this API change is the best way to
shake out frontend bugs.
Note: I added an `llvm_unreachable()` in `bindings/go` and passed in
`nullptr` for the `DebugLoc`. The `llgo` folks will eventually need to
pass a valid `DebugLoc` here.
llvm-svn: 235041
Diffstat (limited to 'llvm/lib/CodeGen/LiveDebugVariables.cpp')
0 files changed, 0 insertions, 0 deletions