[jdk11u-dev] RFR: 8248817: Windows: Improving common cross-platform code

Reka Kovacs duke at openjdk.java.net
Thu Oct 14 17:50:55 UTC 2021


On Sun, 29 Aug 2021 19:03:52 GMT, Reka Kovacs <duke at openjdk.java.net> wrote:

> Not a clean backport. Differences to the original commit:
> 
> - The removed lines in `stubGenerator_x86_32.cpp` and `stubGenerator_x86_64.cpp` are slightly different on tip (comments - [JDK-8234563](https://bugs.openjdk.java.net/browse/JDK-8234563), MP checks - [JDK-8188764](https://bugs.openjdk.java.net/browse/JDK-8188764)), but fundamentally the same.
> - Changes to `add`, `xchg` and `cmpxchg` definitions in `atomic_windows_x86.hpp` needed to be adjusted to keep the original parameter orders (changed in JDK 14 by [JDK-8234563](https://bugs.openjdk.java.net/browse/JDK-8234563)).
> 
> Changes to `os_windows_x86.cpp` and `os_windows_x86.hpp` could be applied cleanly.
> 
> This is part of the Windows/AArch64 port.

Sorry for the confusion - I might have only addressed this under #301:

> Actually, the connection between #274 and #300 is administrative in nature (originally, refactorings of 3 different areas were reviewed under the same JBS bug), there's no real dependency.

You are both right, this patch is technically not needed for adding Windows/AArch64 support. I'm closing the PR.

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

PR: https://git.openjdk.java.net/jdk11u-dev/pull/300


More information about the jdk-updates-dev mailing list