RFR: 8342769: HotSpot Windows/gcc port is broken [v12]

Julian Waters jwaters at openjdk.org
Fri Dec 6 10:39:03 UTC 2024


> Several areas in HotSpot are broken in the gcc port. These, with the exception of 1 rather big oversight within SharedRuntime::frem and SharedRuntime::drem, are all minor correctness issues within the code. These mostly can be fixed with simple changes to the code. Note that I am not sure whether the SharedRuntime::frem and SharedRuntime::drem fix is correct. It may be that they can be removed entirely

Julian Waters 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 22 additional commits since the last revision:

 - Merge branch 'openjdk:master' into hotspot
 - fmod_winarm64 in sharedRuntime.cpp
 - fmod_winarm64 in sharedRuntimeRem.cpp
 - fmod_winarm64 in sharedRuntime.hpp
 - Typo in sharedRuntimeRem.cpp
 - Comment in sharedRuntimeRem.cpp
 - Merge branch 'openjdk:master' into hotspot
 - Reintroduce ifdef removed in the nuking of the Windows 32 bit x86 port in sharedRuntimeRem.cpp
 - fmod_win64 in sharedRuntime.cpp
 - Should only declare fmod_win64 on Windows/ARM64 sharedRuntime.hpp
 - ... and 12 more: https://git.openjdk.org/jdk/compare/4f5aa6c0...027d494b

-------------

Changes:
  - all: https://git.openjdk.org/jdk/pull/21627/files
  - new: https://git.openjdk.org/jdk/pull/21627/files/37e0e4d7..027d494b

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=21627&range=11
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=21627&range=10-11

  Stats: 121630 lines in 1840 files changed: 90091 ins; 23082 del; 8457 mod
  Patch: https://git.openjdk.org/jdk/pull/21627.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/21627/head:pull/21627

PR: https://git.openjdk.org/jdk/pull/21627


More information about the hotspot-dev mailing list