diff options
| author | Jan Kotas <jkotas@microsoft.com> | 2019-02-18 16:37:19 -0800 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2019-02-18 16:37:19 -0800 |
| commit | 281f69036a435d11349fffdee6bb2f0c20f818eb (patch) | |
| tree | 6fd05e3671cc2895ce4de4261bb7d8faf474addf /src/vm/util.cpp | |
| parent | fe4efab4b771d6c01abe1109f7a81e405aa9fa4d (diff) | |
| download | coreclr-281f69036a435d11349fffdee6bb2f0c20f818eb.tar.gz coreclr-281f69036a435d11349fffdee6bb2f0c20f818eb.tar.bz2 coreclr-281f69036a435d11349fffdee6bb2f0c20f818eb.zip | |
Remove resetting of unmanaged thread locale from thread pool APIs (#22682)
The cost of doing this is showing up in the TechEmpower profiles. There is a lot of thread locale state in unmanaged libraries and OS APIs that the threadpool friendly code has to be aware off. It does not make sense to keep reseting just the Locale.
Diffstat (limited to 'src/vm/util.cpp')
| -rw-r--r-- | src/vm/util.cpp | 19 |
1 files changed, 0 insertions, 19 deletions
diff --git a/src/vm/util.cpp b/src/vm/util.cpp index 65fe6dd39c..f03c5e3605 100644 --- a/src/vm/util.cpp +++ b/src/vm/util.cpp @@ -2043,11 +2043,6 @@ size_t GetCacheSizePerLogicalCpu(BOOL bTrueSize) //--------------------------------------------------------------------- #ifndef FEATURE_PAL -ThreadLocaleHolder::~ThreadLocaleHolder() -{ - SetThreadLocale(m_locale); -} - HMODULE CLRGetModuleHandle(LPCWSTR lpModuleFileName) { // Don't use dynamic contract: will override GetLastError value @@ -2058,20 +2053,6 @@ HMODULE CLRGetModuleHandle(LPCWSTR lpModuleFileName) HMODULE hMod = WszGetModuleHandle(lpModuleFileName); return hMod; } - - -HMODULE CLRGetCurrentModuleHandle() -{ - // Don't use dynamic contract: will override GetLastError value - STATIC_CONTRACT_NOTHROW; - STATIC_CONTRACT_GC_NOTRIGGER; - STATIC_CONTRACT_FORBID_FAULT; - - HMODULE hMod = WszGetModuleHandle(NULL); - return hMod; -} - - #endif // !FEATURE_PAL LPVOID EEHeapAllocInProcessHeap(DWORD dwFlags, SIZE_T dwBytes); |