Minor fixes of signed pointers for 32-bit hosts on top of D71498

[jankratochvil]

Summary

This is a similar fix as D71498 just for code with less visible impact.
I haven't tried to execute this code, I have no testcase for it.

I have made a [[ https://people.redhat.com/jkratoch/addr_t2.patch | full class wrapper of addr_t ]], not sure if you think it makes sense to upstream it (after cleaning it up a bit). It disclosed these issues:

• GetCrashReasonString
  • FIX here: reinterpret_cast<lldb::addr_t>
• NativeProcessLinux::ReadMemory
  • safe: reinterpret_cast<void *>
  • safe: (void *)addr
• NativeProcessLinux::WriteMemory
  • safe: (void *)addr
• Value::GetValueAsData
  • safe: reinterpret_cast<uint8_t *>(address)
• ValueObject::GetPointeeData
  • safe: (uint8_t *)(addr + offset)
• Type::ReadFromMemory
  • safe: reinterpret_cast<uint8_t *>(addr)
• IRMemoryMap::Malloc
  • FIX here: (uint64_t)(lldb::addr_t)process_sp.get()
• IRExecutionUnit::GetRunnableInfo
  • FIX by D71498: (lldb::addr_t)fun_ptr

[jankratochvil]

I will merge this with D71498.

lldb/source/Expression/IRMemoryMap.cpp

if (allocation_address == LLDB_INVALID_ADDRESS) {
       return LLDB_INVALID_ADDRESS;
     }
     break;
   case eAllocationPolicyMirror:
     process_sp = m_process_wp.lock();
     LLDB_LOGF(log,
               "IRMemoryMap::%s process_sp=0x%" PRIx64
               ", process_sp->CanJIT()=%s, process_sp->IsAlive()=%s",
-              __FUNCTION__, (lldb::addr_t)process_sp.get(),
+              __FUNCTION__, (uint64_t)(uintptr_t)process_sp.get(),
               process_sp && process_sp->CanJIT() ? "true" : "false",
               process_sp && process_sp->IsAlive() ? "true" : "false");
     if (process_sp && process_sp->CanJIT() && process_sp->IsAlive()) {
       if (!zero_memory)
         allocation_address =
             process_sp->AllocateMemory(allocation_size, permissions, error);
       else
         allocation_address =

lldb/source/Plugins/Process/POSIX/CrashReason.cpp

 
 std::string GetCrashReasonString(CrashReason reason, const siginfo_t &info) {
   std::string str;
 
 // make sure that siginfo_t has the bound fields available.
 #if defined(si_lower) && defined(si_upper)
   if (reason == CrashReason::eBoundViolation) {
     str = "signal SIGSEGV";
-    AppendBounds(str, reinterpret_cast<lldb::addr_t>(info.si_lower),
-                 reinterpret_cast<lldb::addr_t>(info.si_upper),
-                 reinterpret_cast<lldb::addr_t>(info.si_addr));
+    AppendBounds(str, (lldb::addr_t)(uintptr_t)info.si_lower,
+                 (lldb::addr_t)(uintptr_t)info.si_upper,
+                 (lldb::addr_t)(uintptr_t)info.si_addr);
     return str;
   }
 #endif
 
-  return GetCrashReasonString(reason,
-                              reinterpret_cast<lldb::addr_t>(info.si_addr));
+  return GetCrashReasonString(reason, (lldb::addr_t)(uintptr_t)info.si_addr);
 }
 
 std::string GetCrashReasonString(CrashReason reason, lldb::addr_t fault_addr) {
   std::string str;
 
   switch (reason) {
   default:
     str = "unknown crash reason";