Integrated: 8311064: Windows builds fail without precompiled headers after JDK-8310728
Daniel Jeliński
djelinski at openjdk.org
Thu Jun 29 06:09:01 UTC 2023
On Wed, 28 Jun 2023 21:13:50 GMT, Daniel Jeliński <djelinski at openjdk.org> wrote:
> Please review this change to fix Windows build without precompiled headers.
>
> After JDK-8310728:
>> the compiler enforces the C++11 requirement that all functions declared inline must have a definition available in the same translation unit if they're used
>
> When precompiled headers are used, more headers are automatically included, which allowed the build to pass without this change.
>
> This PR adds 2 includes and un-inlines `PlatformMutex` destructor, which was needed by 3 files and not easily exposed.
>
> Windows build passes with this change, both with and without precompiled headers. Full tier1-5 build in progress.
This pull request has now been integrated.
Changeset: af319d96
Author: Daniel Jeliński <djelinski at openjdk.org>
URL: https://git.openjdk.org/jdk/commit/af319d963f180a23a0a9acdc9a4972f90ee586e0
Stats: 10 lines in 4 files changed: 5 ins; 4 del; 1 mod
8311064: Windows builds fail without precompiled headers after JDK-8310728
Reviewed-by: dholmes
-------------
PR: https://git.openjdk.org/jdk/pull/14701
More information about the hotspot-dev
mailing list