diff options
author | Pavel Labath <pavel@labath.sk> | 2019-03-05 12:51:20 +0000 |
---|---|---|
committer | Pavel Labath <pavel@labath.sk> | 2019-03-05 12:51:20 +0000 |
commit | a55999301e2169905ff2359fa17584df05378a01 (patch) | |
tree | df4890a4d9a86d86e39b049b47abd8e7b6f20e82 | |
parent | 826624f00a8c2fb5b65830aeb421d8fe38c5ea0e (diff) | |
download | bcm5719-llvm-a55999301e2169905ff2359fa17584df05378a01.tar.gz bcm5719-llvm-a55999301e2169905ff2359fa17584df05378a01.zip |
One more UserIDResolver fix
The intention in r355323 has been to implement a no-op resolver in the
HostInfoBase class, which will then be shadowed a an implementation in
the HostInfoPosix class. However, I add the shadowing declaration in
HostInfoPosix.h, and instead had implemented the HostInfoBase function
in HostInfoPosix.cpp. This has lead to undefined symbols on windows, and
a subsequent implementation of a no-op resolver in HostInfoWindows
(r355329).
Since now there is no point on having a no-op resolver in the base
class, I just remove the base declaration altogether, and have
HostInfoPosix implement the (newly-declared) HostInfoPosix version of
that function.
llvm-svn: 355398
-rw-r--r-- | lldb/include/lldb/Host/HostInfoBase.h | 2 | ||||
-rw-r--r-- | lldb/include/lldb/Host/posix/HostInfoPosix.h | 2 | ||||
-rw-r--r-- | lldb/source/Host/posix/HostInfoPosix.cpp | 2 |
3 files changed, 3 insertions, 3 deletions
diff --git a/lldb/include/lldb/Host/HostInfoBase.h b/lldb/include/lldb/Host/HostInfoBase.h index f2370f1d00a..f322012906b 100644 --- a/lldb/include/lldb/Host/HostInfoBase.h +++ b/lldb/include/lldb/Host/HostInfoBase.h @@ -99,8 +99,6 @@ public: //--------------------------------------------------------------------------- static ArchSpec GetAugmentedArchSpec(llvm::StringRef triple); - static UserIDResolver &GetUserIDResolver(); - protected: static bool ComputeSharedLibraryDirectory(FileSpec &file_spec); static bool ComputeSupportExeDirectory(FileSpec &file_spec); diff --git a/lldb/include/lldb/Host/posix/HostInfoPosix.h b/lldb/include/lldb/Host/posix/HostInfoPosix.h index 45e86e22b10..a3fbe49560c 100644 --- a/lldb/include/lldb/Host/posix/HostInfoPosix.h +++ b/lldb/include/lldb/Host/posix/HostInfoPosix.h @@ -35,6 +35,8 @@ public: static bool ComputePathRelativeToLibrary(FileSpec &file_spec, llvm::StringRef dir); + static UserIDResolver &GetUserIDResolver(); + protected: static bool ComputeSupportExeDirectory(FileSpec &file_spec); static bool ComputeHeaderDirectory(FileSpec &file_spec); diff --git a/lldb/source/Host/posix/HostInfoPosix.cpp b/lldb/source/Host/posix/HostInfoPosix.cpp index 45c8151c96e..6e4208dd0ca 100644 --- a/lldb/source/Host/posix/HostInfoPosix.cpp +++ b/lldb/source/Host/posix/HostInfoPosix.cpp @@ -106,7 +106,7 @@ llvm::Optional<std::string> PosixUserIDResolver::DoGetGroupName(id_t gid) { static llvm::ManagedStatic<PosixUserIDResolver> g_user_id_resolver; -UserIDResolver &HostInfoBase::GetUserIDResolver() { +UserIDResolver &HostInfoPosix::GetUserIDResolver() { return *g_user_id_resolver; } |