On Tue, 28 Apr 2026 01:41:54 GMT, Jaikiran Pai <[email protected]> wrote:
>> Can I please get a review of this change which removes support for the >> `-noclassgc`, `-noverify`, `-verifyremote` options from the `java` launcher >> and the `-Xverify:none` option from the VM? >> >> These options have been deprecated since some releases and have been >> printing a deprecation warning. There were a few more `java` launcher >> options which were deprecated for removal a while back through >> https://bugs.openjdk.org/browse/JDK-8340244. Those remaining ones will >> continue to stay deprecated but will be removed in some future version, >> given that they are relatively more commonly used. >> >> The CSR for this change is available for review >> https://bugs.openjdk.org/browse/JDK-8382727 and has additional details about >> this change. >> >> The `test/hotspot/jtreg/gc/shenandoah/compiler/CallMultipleCatchProjs.java` >> test had to be updated to remove its usage of `-Xverify:none`. That test was >> introduced in https://bugs.openjdk.org/browse/JDK-8231405, but I haven't >> been able to locate a RFR thread for that (I checked hotspot-gc-dev, >> shenandoah-dev mailing lists from September 2019), so it's not clear to me >> why the `-Xverify:none` was needed in that test. A brief look at that test >> suggests that this option may not be necessary, so I've removed it. Please >> take a look and let me know if that option was intentional (in which case we >> need to decide how we deal with that test with this option being removed). >> >> tier1, tier2, tier3 testing has completed without any related failures with >> this change. I plan to run higher tier tests in the coming days. >> >> --------- >> - [x] I confirm that I make this contribution in accordance with the >> [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai). > > Jaikiran Pai 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 five additional > commits since the last revision: > > - merge latest from master branch > - merge latest from master branch > - remove mention of -Xverify:none from the java launcher help text > - remove -Xverify:none from a test > - 8373481: Remove the deprecated -noclassgc, -noverify, -Xverify:none and > -verifyremote options from the java launcher Thank you for all the reviews. I'll go ahead and integrate this in the next few hours. For the `-Xverify:none` removal from the `test/hotspot/jtreg/gc/shenandoah/compiler/CallMultipleCatchProjs.java` test, I haven't heard from the original author of that test. But my brief look at that test and also Aleksey's input, I think the change is OK (the test doesn't do any bytecode handling/fiddling). Plus it continues to pass with the change. ------------- PR Comment: https://git.openjdk.org/jdk/pull/30877#issuecomment-4334161366
