[Lldb-commits] [PATCH] D150366: [lldb][NFCI] Use llvm's libDebugInfo for DebugRanges

2023-05-23 Thread Felipe de Azevedo Piovezan via Phabricator via lldb-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG70aad4ec90f2: [lldb][NFCI] Use llvms libDebugInfo for DebugRanges (authored by fdeazeve). Changed prior to commit: https://reviews.llvm.org/D150366?vs=521300=524731#toc Repository: rG LLVM Github

[Lldb-commits] [PATCH] D150366: [lldb][NFCI] Use llvm's libDebugInfo for DebugRanges

2023-05-15 Thread Jonas Devlieghere via Phabricator via lldb-commits
JDevlieghere accepted this revision. JDevlieghere added a comment. This revision is now accepted and ready to land. LGTM Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D150366/new/ https://reviews.llvm.org/D150366

[Lldb-commits] [PATCH] D150366: [lldb][NFCI] Use llvm's libDebugInfo for DebugRanges

2023-05-12 Thread Felipe de Azevedo Piovezan via Phabricator via lldb-commits
fdeazeve added a comment. In D150366#4336168 , @bulbazord wrote: > Ok, this looks like it's doing the same thing to me which is good. My > understanding of this change is that you're changing > `lldb::DWARFDebugRanges::Extract` to use

[Lldb-commits] [PATCH] D150366: [lldb][NFCI] Use llvm's libDebugInfo for DebugRanges

2023-05-11 Thread Alex Langford via Phabricator via lldb-commits
bulbazord added a comment. Ok, this looks like it's doing the same thing to me which is good. My understanding of this change is that you're changing `lldb::DWARFDebugRanges::Extract` to use `llvm::DWARFDebugRangeList` instead of `lldb::DWARFRangeList`. Out of curiosity, do you have an idea

[Lldb-commits] [PATCH] D150366: [lldb][NFCI] Use llvm's libDebugInfo for DebugRanges

2023-05-11 Thread Felipe de Azevedo Piovezan via Phabricator via lldb-commits
fdeazeve created this revision. Herald added a project: All. fdeazeve requested review of this revision. Herald added a project: LLDB. Herald added a subscriber: lldb-commits. In an effort to unify the different dwarf parsers available in the codebase, this commit removes LLDB's custom parsing