RFR: JDK-8303166: Disable RTTI for Hotspot when building with MSVC [v3]
Alex Menkov
amenkov at openjdk.org
Sat Feb 25 01:51:03 UTC 2023
On Fri, 24 Feb 2023 19:56:39 GMT, Justin King <jcking at openjdk.org> wrote:
>> Disable RTTI `-GR-` for Hotspot when building with MSVC. This drops the size of `jvm.dll` by roughly 1 MB. Hotspot does not rely on RTTI and it is disabled for both GCC/Clang and Open XL C/C++ already. This change disables it for MSVC, ensuring we do not accidently rely on RTTI for Windows-specific code while also decreasing the resulting binary size.
>>
>> Cheers to @alexmenkov for finding the tricky root cause of why disabling RTTI was causing a subset of serviceability agent tests to fail.
>
> Justin King has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains four additional commits since the last revision:
>
> - Merge remote-tracking branch 'upstream/master' into msvc-rtti
> - Update src/hotspot/share/runtime/notificationThread.hpp
>
> Co-authored-by: Chris Plummer <chris.plummer at oracle.com>
> - Force MSVC to emit unique vtable for NotificationThread
>
> Signed-off-by: Justin King <jcking at google.com>
> - Disable RTTI with MSVC
>
> Signed-off-by: Justin King <jcking at google.com>
Need to update copyright year in flags-cflags.m4 and notificationThread.hpp
-------------
Marked as reviewed by amenkov (Reviewer).
PR: https://git.openjdk.org/jdk/pull/12743
More information about the build-dev
mailing list