RFR: JDK-8280583: Always build NMT [v2]
Thomas Stuefe
stuefe at openjdk.java.net
Wed Jan 26 06:23:41 UTC 2022
> After discussing this on hotspot-runtime-dev [1], the general opinion seems to be that it would be worthwhile to get rid of INCLUDE_NMT and make NMT unconditional. This affects minimal builds only. As pointed out in the mail thread, the overhead is very small and it would get rid of one configuration to build and test.
>
> [1] https://mail.openjdk.java.net/pipermail/hotspot-runtime-dev/2022-January/053504.html
>
> Patch removes INCLUDE_NMT from hotspot, as well as dependent macros, as well as the associated build option.
Thomas Stuefe has updated the pull request incrementally with 15 additional commits since the last revision:
- 8280377: MethodHandleProxies does not correctly invoke default methods with varags
Reviewed-by: alanb
- 8213905: reflection not working for type annotations applied to exception types in the inner class constructor
Reviewed-by: jlahoda
- 8279242: Reflection newInstance() error message when constructor has no access modifiers could use improvement
Reviewed-by: iris, dholmes, mchung
- 8269542: JDWP: EnableCollection support is no longer spec compliant after JDK-8255987
8258071: Fix for JDK-8255987 can be subverted with ObjectReference.EnableCollection
Reviewed-by: dholmes, pliden
- 8280166: Extend java/lang/instrument/GetObjectSizeIntrinsicsTest.java test cases
Reviewed-by: sspitsyn, lmesnik
- 8280041: Retry loop issues in java.io.ClassCache
Co-authored-by: Peter Levart <plevart at openjdk.org>
Reviewed-by: rkennke, rriggs, plevart
- 8280168: Add Objects.toIdentityString
Reviewed-by: alanb, mchung, rriggs, smarks
- 8279946: (ch) java.nio.channels.FileChannel tryLock and write methods are missing @throws NonWritableChannelException
Reviewed-by: alanb
- 8280396: G1: Full gc mark stack draining should prefer to make work available to other threads
Reviewed-by: sjohanss, ayang
- 8280414: Memory leak in DefaultProxySelector
Reviewed-by: dfuchs
- ... and 5 more: https://git.openjdk.java.net/jdk/compare/a19176e6...44478392
-------------
Changes:
- all: https://git.openjdk.java.net/jdk/pull/7213/files
- new: https://git.openjdk.java.net/jdk/pull/7213/files/a19176e6..44478392
Webrevs:
- full: https://webrevs.openjdk.java.net/?repo=jdk&pr=7213&range=01
- incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=7213&range=00-01
Stats: 948 lines in 56 files changed: 717 ins; 90 del; 141 mod
Patch: https://git.openjdk.java.net/jdk/pull/7213.diff
Fetch: git fetch https://git.openjdk.java.net/jdk pull/7213/head:pull/7213
PR: https://git.openjdk.java.net/jdk/pull/7213
More information about the build-dev
mailing list