[jdk11u-dev] RFR: 8254072: AArch64: Get rid of --disable-warnings-as-errors on Windows+ARM64 build [v3]
Reka Kovacs
rkovacs at openjdk.java.net
Fri Feb 11 00:55:47 UTC 2022
> Despite its title, this patch only touches common code that's present before adding Windows/AArch64 support. Not a clean backport. Differences to the original commit:
>
> - This is the first time we disable an MSVC warning for hotspot in 11, so in `make/hotspot/lib/CompileJvm.gmk` this introduces a new `DISABLED_WARNINGS_microsoft` line.
> - We disable the same warning in `make/hotspot/lib/CompileGtest.gmk`.
> - A tiny useless switch is removed from `src/hotspot/cpu/aarch64/aarch64.ad` that's no longer present on tip (probably not worth a separate PR).
> - <s>Changes to `src/hotspot/cpu/aarch64/frame_aarch64.cpp` don't yet include the changes from #215.</s>
> - The assert on line 1477 in `src/hotspot/cpu/aarch64/sharedRuntime_aarch64.cpp` needed some adjustment to compile. (I'm not sure how it compiles on tip, as I don't see `-Wsign-compare` turned off there either, but the types seem to be the same.)
>
> This is part of the Windows/AArch64 port.
Reka Kovacs has updated the pull request incrementally with three additional commits since the last revision:
- Use size_t in generate_disjoint_copy
- Explain disabled MSVC warning C4146 in comment
- Fix MSVC warning C4334: result of 32-bit shift implicitly converted to 64 bits
-------------
Changes:
- all: https://git.openjdk.java.net/jdk11u-dev/pull/280/files
- new: https://git.openjdk.java.net/jdk11u-dev/pull/280/files/d7975df8..09587d5e
Webrevs:
- full: https://webrevs.openjdk.java.net/?repo=jdk11u-dev&pr=280&range=02
- incr: https://webrevs.openjdk.java.net/?repo=jdk11u-dev&pr=280&range=01-02
Stats: 10 lines in 4 files changed: 5 ins; 0 del; 5 mod
Patch: https://git.openjdk.java.net/jdk11u-dev/pull/280.diff
Fetch: git fetch https://git.openjdk.java.net/jdk11u-dev pull/280/head:pull/280
PR: https://git.openjdk.java.net/jdk11u-dev/pull/280
More information about the jdk-updates-dev
mailing list