[Lldb-commits] [PATCH] D141330: [lldb] Limit 8b259fe573e1 to dSYMs

2023-01-12 Thread Pavel Labath via Phabricator via lldb-commits
labath added a comment. Just to clarify: I was responding to the question about the ELF file usage, not questioning the overall approach. Given that MachO has a special code for dsym files, I think it makes sense to use it. In D141330#4043807 , @jasonmo

[Lldb-commits] [PATCH] D141330: [lldb] Limit 8b259fe573e1 to dSYMs

2023-01-11 Thread Jason Molenda via Phabricator via lldb-commits
jasonmolenda added a comment. In D141330#4043397 , @labath wrote: > In D141330#4037932 , @JDevlieghere > wrote: > >> In D141330#4037925 , @aprantl >> wrote: >> >>> Shoul

[Lldb-commits] [PATCH] D141330: [lldb] Limit 8b259fe573e1 to dSYMs

2023-01-11 Thread Pavel Labath via Phabricator via lldb-commits
labath added a comment. In D141330#4037932 , @JDevlieghere wrote: > In D141330#4037925 , @aprantl wrote: > >> Should this be true for a fully linked ELF executable, too? > > Sounds reasonable, but adding @labath

[Lldb-commits] [PATCH] D141330: [lldb] Limit 8b259fe573e1 to dSYMs

2023-01-09 Thread Jonas Devlieghere via Phabricator via lldb-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG9b737f148d88: [lldb] Limit trusting aranges to dSYMs only. (authored by JDevlieghere). Herald added a project: LLDB. Herald added a subscriber: lldb-commits. Changed prior to commit: https://reviews.llv