Fix load address resolution on Windows

[zturner]

Summary

When loading a module as a result of the "target create" command, there is the potential for ObjectFilePECOFF to stomp the triple. This can confuse later code which depends on the triple. Ultimately this fixes load address resolution on Windows because the correct triple is plumbed all the way through, so DynamicLoaderWindows can now correctly recognize that it should be used.

[clayborg]

Close. A couple changed as annotated in the source. There are two categories of unknown:

• specified "unknown" for the vendor or OS where the user actually typed "unknown" in the triple
• unspecified "unknown" for the vendor or OS where the user didn't specify it.

Add the quick fixes I specify above and this patch is good to go.

source/Core/Module.cpp

1314

We need to make sure the vendor wasn't specified as "unknown" by also calling:

bool ArchSpec::TripleVendorWasSpecified(). So the above line should be:

if (m_arch.GetTriple().getVendor() == llvm::Triple::UnknownVendor && m_arch.TripleVendorWasSpecified())

1316

Save thing as the previous, make sure "unknown" wasn't specified:

if (m_arch.GetTriple().getOS() == llvm::Triple::UnknownOS && m_arch.TripleOSWasSpecified())

[zturner]

Just to be clear, we should be checking that the triple vendor was NOT specified right? The code you wrote checks that it was specified.

Would it be worthwhile to make a function called ArchSpec::MergeFrom(const ArchSpec &other), which basically does what I've done here (with a comment explaining how it works)?

[zturner]

Fixed suggested issues, and moved the logic into a function in ArchSpec, since I thought it might be useful.

I reversed your conditional though because it looked like a thinko.

LMK if this looks good.

[clayborg]

Looks good

source/Core/Module.cpp

if (m_did_load_objfile == false)
                                                    m_object_offset,
                                                    file_size - m_object_offset,
                                                    data_sp,
                                                    data_offset);
             if (m_objfile_sp)
             {
                 // Once we get the object file, update our module with the object file's 
                 // architecture since it might differ in vendor/os if some parts were
-                // unknown.
-                m_objfile_sp->GetArchitecture (m_arch);
+                // unknown.  But since the matching arch might already be more specific
+                // than the generic COFF architecture, only overwrite those fields that
+                // are unknown.
+                ArchSpec new_arch;
+                m_objfile_sp->GetArchitecture (new_arch);
+                if (m_arch.GetTriple().getVendor() == llvm::Triple::UnknownVendor)
+                    m_arch.GetTriple().setVendor(new_arch.GetTriple().getVendor());
+                if (m_arch.GetTriple().getOS() == llvm::Triple::UnknownOS)
+                    m_arch.GetTriple().setOS(new_arch.GetTriple().getOS());
+                if (m_arch.GetTriple().getArch() == llvm::Triple::UnknownArch)
+                    m_arch.GetTriple().setArch(new_arch.GetTriple().getArch());
+                if (m_arch.GetTriple().getEnvironment() == llvm::Triple::UnknownEnvironment)
+                    m_arch.GetTriple().setEnvironment(new_arch.GetTriple().getEnvironment());
             }
             else
             {
                 ReportError ("failed to load objfile for %s", GetFileSpec().GetPath().c_str());
             }
         }
     }
     return m_objfile_sp.get();

source/Plugins/ObjectFile/PECOFF/ObjectFilePECOFF.cpp

if (ObjectFilePECOFF::MagicBytesMatch(data_sp))
             lldb::offset_t offset = dos_header.e_lfanew;
             uint32_t pe_signature = data.GetU32(&offset);
             if (pe_signature != IMAGE_NT_SIGNATURE)
                 return false;
             if (ParseCOFFHeader(data, &offset, coff_header))
             {
                 ArchSpec spec;
                 if (coff_header.machine == MachineAmd64)
+                {
                     spec.SetTriple("x86_64-pc-windows");
+                    specs.Append(ModuleSpec(file, spec));
+                }
                 else if (coff_header.machine == MachineX86)
+                {
                     spec.SetTriple("i386-pc-windows");
                     specs.Append(ModuleSpec(file, spec));
+                    spec.SetTriple("i686-pc-windows");
+                    specs.Append(ModuleSpec(file, spec));
+                }
             }
         }
     }
 
     return specs.GetSize() - initial_count;
 }