[Lldb-commits] [PATCH] D121442: [lldb] Don't overwrite the host arch (from qHostInfo) with the process arch (from qProcessInfo)

Jonas Devlieghere via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Thu Mar 10 22:52:13 PST 2022


JDevlieghere created this revision.
JDevlieghere added reviewers: jasonmolenda, labath, mgorny.
Herald added a project: All.
JDevlieghere requested review of this revision.

Don't overwrite the host architecture (obtained from qHostInfo in GDBRemoteCommunicationClient::GetHostInfo) with the process info (obtained from qProcessInfo in GDBRemoteCommunicationClient::GetCurrentProcessInfo).


https://reviews.llvm.org/D121442

Files:
  lldb/source/Plugins/Process/gdb-remote/GDBRemoteCommunicationClient.cpp


Index: lldb/source/Plugins/Process/gdb-remote/GDBRemoteCommunicationClient.cpp
===================================================================
--- lldb/source/Plugins/Process/gdb-remote/GDBRemoteCommunicationClient.cpp
+++ lldb/source/Plugins/Process/gdb-remote/GDBRemoteCommunicationClient.cpp
@@ -2246,9 +2246,6 @@
         m_process_arch.GetTriple().setVendorName(llvm::StringRef(vendor_name));
         m_process_arch.GetTriple().setOSName(llvm::StringRef(os_name));
         m_process_arch.GetTriple().setEnvironmentName(llvm::StringRef(environment));
-        m_host_arch.GetTriple().setVendorName(llvm::StringRef(vendor_name));
-        m_host_arch.GetTriple().setOSName(llvm::StringRef(os_name));
-        m_host_arch.GetTriple().setEnvironmentName(llvm::StringRef(environment));
       }
       return true;
     }


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D121442.414585.patch
Type: text/x-patch
Size: 828 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20220311/b5a431cd/attachment.bin>


More information about the lldb-commits mailing list