Don't stomp the triple when loading a PECOFF target.
authorZachary Turner <zturner@google.com>
Thu, 22 Jan 2015 18:59:05 +0000 (18:59 +0000)
committerZachary Turner <zturner@google.com>
Thu, 22 Jan 2015 18:59:05 +0000 (18:59 +0000)
commit5e6f45201f0b62c1e7a24fc396f3ea6e10dc880d
tree6cc98b6be3da45381cb35092b3be5d6846dff874
parenta7bdc7e6718219d9585659a5329851a69759626d
Don't stomp the triple when loading a PECOFF target.

When you create a target, it tries to look for the platform's list
of supported architectures for a match.  The match it finds can
contain specific triples, like i386-pc-windows-msvc.  Later, we
overwrite this value with the most generic triple that can apply
to any platform with COFF support, causing some of the fields of
the triple to get overwritten.

This patch changes the behavior to only merge in values from the COFF
triple if the fields of the matching triple were unknown/unspecified
to begin with.

This fixes load address resolution on Windows, since it enables the
DynamicLoaderWindows to be used instead of DynamicLoaderStatic.

Reviewed by: Greg Clayton
Differential Revision: http://reviews.llvm.org/D7120

llvm-svn: 226849
lldb/include/lldb/Core/ArchSpec.h
lldb/source/Core/ArchSpec.cpp
lldb/source/Core/Module.cpp
lldb/source/Plugins/ObjectFile/PECOFF/ObjectFilePECOFF.cpp