Binaryen doesn't update .debug_addr section #3460
Comments
bmeurer
added a commit
to bmeurer/binaryen
that referenced
this issue
Dec 23, 2020
This is a minimal patch to add support for also updating the addresses stored in the .debug_addr DWARF section, which is used to perform address resolution at debug time with debug fission. This is minimal in the sence that it only supports the pre-DWARFv5 .debug_addr format that is emitted by Emscripten currently, and which only consists of a simple list of addresses. Ideally the copy of DWARFYAML in the Binaryen tree should be updated with LLVM ToT at some point, which will provide full support for DWARFv5 index tables, including the new .debug_addr format. Ref: WebAssembly#3460 Ref: emscripten-core/emscripten#13099 Bug: https://crbug.com/1161422
bmeurer
added a commit
to bmeurer/binaryen
that referenced
this issue
Dec 23, 2020
This is a minimal patch to add support for also updating the addresses stored in the .debug_addr DWARF section, which is used to perform address resolution at debug time with debug fission. This is minimal in the sence that it only supports the pre-DWARFv5 .debug_addr format that is emitted by Emscripten currently, and which only consists of a simple list of addresses. Ideally the copy of DWARFYAML in the Binaryen tree should be updated with LLVM ToT at some point, which will provide full support for DWARFv5 index tables, including the new .debug_addr format. Ref: WebAssembly#3460 Ref: emscripten-core/emscripten#13099 Bug: https://crbug.com/1161422
bmeurer
added a commit
to bmeurer/binaryen
that referenced
this issue
Dec 23, 2020
This is a minimal patch to add support for also updating the addresses stored in the .debug_addr DWARF section, which is used to perform address resolution at debug time with debug fission. This is minimal in the sence that it only supports the pre-DWARFv5 .debug_addr format that is emitted by Emscripten currently, and which only consists of a simple list of addresses. Ideally the copy of DWARFYAML in the Binaryen tree should be updated with LLVM ToT at some point, which will provide full support for DWARFv5 index tables, including the new .debug_addr format. Ref: WebAssembly#3460 Ref: emscripten-core/emscripten#13099 Bug: https://crbug.com/1161422
|
From the discussion on the linked PR, it sounds like there is no way to correctly update this section, so I'll close this issue. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Downstream issue: emscripten-core/emscripten#13099
Chromium issue: https://crbug.com/1161422
Repro is on the downstream issue, the problem is that
.debug_addrisn't updated by the DWARF support in Binaryen.The text was updated successfully, but these errors were encountered: