Merge pull request #8271 from adrian-prantl/cherry-pick-stable-20230725-Improve-and-modernize-logging-for-Process-CompleteAttach-82717

[Cherry-pick into stable/20230725] Improve and modernize logging for Process::CompleteAttach() (llvm#82717)

Author: adrian-prantl

lldb/source/Target/Process.cpp

@@ -3091,14 +3091,11 @@ void Process::CompleteAttach() {
   DidAttach(process_arch);
 
   if (process_arch.IsValid()) {
+    LLDB_LOG(log,
+             "Process::{0} replacing process architecture with DidAttach() "
+             "architecture: \"{1}\"",
+             __FUNCTION__, process_arch.GetTriple().getTriple());
     GetTarget().SetArchitecture(process_arch);
-    if (log) {
-      const char *triple_str = process_arch.GetTriple().getTriple().c_str();
-      LLDB_LOGF(log,
-                "Process::%s replacing process architecture with DidAttach() "
-                "architecture: %s",
-                __FUNCTION__, triple_str ? triple_str : "<null>");
-    }
   }
 
   // We just attached.  If we have a platform, ask it for the process