Fix reading from file cache using address instead of resolved address for getting a section

[augusto2112]

Summary

This fiex a small problem with https://reviews.llvm.org/D100338 where we'd grab the section from the address instead of resolved address, which could incur a memory read instead of a file cache read if the two addressess were different

[jasonmolenda]

Ach, I should have noticed that during review.

lldb/source/Target/Target.cpp

if (!addr.IsSectionOffset()) {
     }
   }
   if (!resolved_addr.IsValid())
     resolved_addr = addr;
 
   bool is_readonly = false;
   // Read from file cache if read-only section.
   if (!force_live_memory && resolved_addr.IsSectionOffset()) {
-    SectionSP section_sp(addr.GetSection());
+    SectionSP section_sp(resolved_addr.GetSection());
     if (section_sp) {
       auto permissions = Flags(section_sp->GetPermissions());
       is_readonly = !permissions.Test(ePermissionsWritable) &&
                     permissions.Test(ePermissionsReadable);
     }
     if (is_readonly) {
       bytes_read = ReadMemoryFromFileCache(resolved_addr, dst, dst_len, error);
       if (bytes_read > 0)