[Lldb-commits] [lldb] [lldb] Track the target's executable instead of guessing (PR #228131)

[Alex Langford via lldb-commits]

================
@@ -80,8 +80,8 @@ addr_t DYLDRendezvous::ResolveRendezvousAddress() {
   // local object file can help us find it.
   if (info_location == LLDB_INVALID_ADDRESS) {
     Target *target = &m_process->GetTarget();
-    if (target) {
-      ObjectFile *obj_file = target->GetExecutableModule()->GetObjectFile();
+    if (ModuleSP executable_sp = target->GetExecutableModule()) {
----------------
bulbazord wrote:

This drops the null-check for `target`. I assume that's alright since we already have a `m_process`... but why are we pulling a raw target out of a shared pointer anyway?

https://github.com/llvm/llvm-project/pull/228131