Skip to content

Commit

Permalink
8311064: Windows builds fail without precompiled headers after JDK-83…
Browse files Browse the repository at this point in the history
…10728

Reviewed-by: dholmes
  • Loading branch information
djelinski committed Jun 29, 2023
1 parent cbf418a commit af319d9
Show file tree
Hide file tree
Showing 4 changed files with 6 additions and 5 deletions.
2 changes: 1 addition & 1 deletion src/hotspot/os/windows/gc/z/zVirtualMemory_windows.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@
#include "gc/z/zLargePages.inline.hpp"
#include "gc/z/zMapper_windows.hpp"
#include "gc/z/zSyscall_windows.hpp"
#include "gc/z/zVirtualMemory.hpp"
#include "gc/z/zVirtualMemory.inline.hpp"
#include "utilities/align.hpp"
#include "utilities/debug.hpp"

Expand Down
4 changes: 4 additions & 0 deletions src/hotspot/os/windows/os_windows.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -5451,6 +5451,10 @@ void Parker::unpark() {
SetEvent(_ParkHandle);
}

PlatformMutex::~PlatformMutex() {
DeleteCriticalSection(&_mutex);
}

// Platform Monitor implementation

// Must already be locked
Expand Down
4 changes: 0 additions & 4 deletions src/hotspot/os/windows/os_windows.inline.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -65,10 +65,6 @@ inline PlatformMutex::PlatformMutex() {
InitializeCriticalSection(&_mutex);
}

inline PlatformMutex::~PlatformMutex() {
DeleteCriticalSection(&_mutex);
}

inline PlatformMonitor::PlatformMonitor() {
InitializeConditionVariable(&_cond);
}
Expand Down
1 change: 1 addition & 0 deletions src/hotspot/share/gc/z/zPhysicalMemory.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,7 @@
#include "gc/z/zArray.inline.hpp"
#include "gc/z/zGlobals.hpp"
#include "gc/z/zLargePages.inline.hpp"
#include "gc/z/zList.inline.hpp"
#include "gc/z/zNUMA.inline.hpp"
#include "gc/z/zPhysicalMemory.inline.hpp"
#include "logging/log.hpp"
Expand Down

1 comment on commit af319d9

@openjdk-notifier
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please sign in to comment.