[Lldb-commits] [lldb] r214667 - Change ProcessGDBRemote::DidLaunchOrAttach to

Jason Molenda jmolenda at apple.com
Sun Aug 3 14:42:52 PDT 2014


Author: jmolenda
Date: Sun Aug  3 16:42:52 2014
New Revision: 214667

URL: http://llvm.org/viewvc/llvm-project?rev=214667&view=rev
Log:
Change ProcessGDBRemote::DidLaunchOrAttach to 
call Target::SetArchitecture instead of modifying a
reference to the target's architecture so that the
target logging can show that the arch has been changed.

Modified:
    lldb/trunk/source/Plugins/Process/gdb-remote/ProcessGDBRemote.cpp

Modified: lldb/trunk/source/Plugins/Process/gdb-remote/ProcessGDBRemote.cpp
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Plugins/Process/gdb-remote/ProcessGDBRemote.cpp?rev=214667&r1=214666&r2=214667&view=diff
==============================================================================
--- lldb/trunk/source/Plugins/Process/gdb-remote/ProcessGDBRemote.cpp (original)
+++ lldb/trunk/source/Plugins/Process/gdb-remote/ProcessGDBRemote.cpp Sun Aug  3 16:42:52 2014
@@ -1048,7 +1048,7 @@ ProcessGDBRemote::DidLaunchOrAttach (Arc
                 if (process_arch.GetMachine() == llvm::Triple::arm &&
                     process_arch.GetTriple().getVendor() == llvm::Triple::Apple)
                 {
-                    target_arch = process_arch;
+                    GetTarget().SetArchitecture (process_arch);
                 }
                 else
                 {
@@ -1073,7 +1073,7 @@ ProcessGDBRemote::DidLaunchOrAttach (Arc
             {
                 // The target doesn't have a valid architecture yet, set it from
                 // the architecture we got from the remote GDB server
-                target_arch = process_arch;
+                GetTarget().SetArchitecture (process_arch);
             }
         }
     }





More information about the lldb-commits mailing list