[Lldb-commits] [PATCH] D65789: A more robust way of testing debug_line parser near the end of module

2019-08-07 Thread Pavel Labath via Phabricator via lldb-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL368154: A more robust way of testing debug_line parser near the end of module (authored by labath, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Repository: rL

[Lldb-commits] [PATCH] D65789: A more robust way of testing debug_line parser near the end of module

2019-08-07 Thread Pavel Labath via Phabricator via lldb-commits
labath updated this revision to Diff 213845. labath added a comment. - remove output section addresses CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65789/new/ https://reviews.llvm.org/D65789 Files: lit/SymbolFile/DWARF/Inputs/debug-line-basic.script

[Lldb-commits] [PATCH] D65789: A more robust way of testing debug_line parser near the end of module

2019-08-06 Thread Fangrui Song via Phabricator via lldb-commits
MaskRay accepted this revision. MaskRay added inline comments. This revision is now accepted and ready to land. Comment at: lit/SymbolFile/DWARF/Inputs/debug-line-basic.script:6 + .shstrtab : { *(.shstrtab) } + .debug_info0 : { *(.debug_info ) } + .debug_line0 : {

[Lldb-commits] [PATCH] D65789: A more robust way of testing debug_line parser near the end of module

2019-08-06 Thread Pavel Labath via Phabricator via lldb-commits
labath created this revision. labath added a reviewer: MaskRay. While removing -z separate-code makes lld produce place the code at the end of a segment right now, it's possible that future changes to the linker will change that, thereby removing the coverage for the changes in r367983. This