On Fri, 24 Feb 2023 19:56:39 GMT, Justin King <[email protected]> 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 <[email protected]> > - Force MSVC to emit unique vtable for NotificationThread > > Signed-off-by: Justin King <[email protected]> > - Disable RTTI with MSVC > > Signed-off-by: Justin King <[email protected]> The workaround should have been done under JDK-8302817. This issue should have simply disabled RTTI in the build system. ------------- PR: https://git.openjdk.org/jdk/pull/12743
