[Lldb-commits] [PATCH] D127889: [LLDB] Handle DIE with DW_AT_low_pc and empty ranges

2022-06-24 Thread Pavel Labath via Phabricator via lldb-commits
labath added a comment. It should be sufficient to replace the %clang line with the appropriate ld.lld equivalent, and replace REQUIRES: system-linux with REQUIRES: lld. Also it test/Shell/SymbolFile/DWARF would be a better place for this test (and it also includes some tests that you can take

[Lldb-commits] [PATCH] D127889: [LLDB] Handle DIE with DW_AT_low_pc and empty ranges

2022-06-23 Thread Alexander Yermolovich via Phabricator via lldb-commits
ayermolo added a comment. In D127889#3604421 , @DavidSpickett wrote: > I've reverted this due to failures on our bots: > https://lab.llvm.org/buildbot/#/builders/96/builds/25032/steps/6/logs/stdio > > /usr/bin/ld: >

[Lldb-commits] [PATCH] D127889: [LLDB] Handle DIE with DW_AT_low_pc and empty ranges

2022-06-23 Thread David Spickett via Phabricator via lldb-commits
DavidSpickett added a comment. I've reverted this due to failures on our bots: https://lab.llvm.org/buildbot/#/builders/96/builds/25032/steps/6/logs/stdio /usr/bin/ld:

[Lldb-commits] [PATCH] D127889: [LLDB] Handle DIE with DW_AT_low_pc and empty ranges

2022-06-22 Thread Alexander Yermolovich via Phabricator via lldb-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG130167ed1eff: [LLDB] Handle DIE with DW_AT_low_pc and empty ranges (authored by ayermolo). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D127889/new/

[Lldb-commits] [PATCH] D127889: [LLDB] Handle DIE with DW_AT_low_pc and empty ranges

2022-06-15 Thread Alexander Yermolovich via Phabricator via lldb-commits
ayermolo created this revision. Herald added subscribers: hoy, modimo, wenlei. Herald added a project: All. ayermolo requested review of this revision. Herald added a reviewer: jdoerfert. Herald added subscribers: lldb-commits, sstefan1. Herald added a project: LLDB. The case comes out of how