diff options
| author | mlugg <mlugg@mlugg.co.uk> | 2024-03-25 03:51:58 +0000 |
|---|---|---|
| committer | Matthew Lugg <mlugg@mlugg.co.uk> | 2024-03-25 15:05:52 +0000 |
| commit | 31a7f22b800c091962726de2dd29f10a8eb25b78 (patch) | |
| tree | 2cfbb25b0ed73eecb17c976777df08765237a4be /lib/std/debug.zig | |
| parent | 5c628312b16ce972cc3108ed44eed47960e17af7 (diff) | |
| download | zig-31a7f22b800c091962726de2dd29f10a8eb25b78.tar.gz zig-31a7f22b800c091962726de2dd29f10a8eb25b78.zip | |
llvm: update current debug location scope when entering debug scope
This issue was causing debug information to sometimes not function
correctly for some local variables, with debuggers simply reporting that
the variable does not exist. What was happening was that after an AIR
body - and thus debug lexical scope - begins, but before any `dbg_stmt`
within it, the `scope` on `self.wip.debug_location` refers to the parent
scope, but the `scope` field on the `DILocalVariable` metadata passed to
`@llvm.dbg.declare` points, correctly, to the nested scope. I haven't
looked into precisely what happens here, but in short, it would appear
that LLVM Doesn't Like It (tm).
The fix is simple: when we change `self.scope` at the start or end of an
AIR body, also modify the scope on `self.wip.debug_location`. This is
correct as we always want the debug info for an instruction to be
associated with the block it is within, even if the line/column are
slightly outdated for any reason.
Diffstat (limited to 'lib/std/debug.zig')
0 files changed, 0 insertions, 0 deletions
