On Fri, 8 May 2026 10:27:49 GMT, Thomas Schatzl <[email protected]> wrote:
>> Hi all, >> >> please review this change that allows building of the JDK without Serial >> GC enabled. >> >> There were some hardcoded entries for -XX:+UseSerialGC in the makefiles, and >> one of the gtests is serial gc specific and needed to be moved. Noticed >> during tests with JEP 523: Make G1 the Default Garbage Collector in All >> Environments. >> >> I opted to a different fix than suggested in the CR, instead of wholesale >> ignoring unrecognized options, only add serial gc to the relevant options if >> the feature is enabled in the build. The change seems small enough. >> >> Tested manually with and without `--with-jvm-features=-serialgc`, both image >> building and running the test works in both cases (being skipped/not found >> if serial gc has not been enabled). >> >> Other testing: gha >> >> Thanks, >> Thomas >> >> >> >> >> --------- >> - [x] I confirm that I make this contribution in accordance with the >> [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai). > > Thomas Schatzl has updated the pull request incrementally with one additional > commit since the last revision: > > * use the default collector, do not force Serial GC for anything >👍 On this change. Very nice that we can do this without degradation to the >build times. > >Should the issue title reflect that this is now just "use the default GC when >building"? No. There is still the issue with the serial-gc specific test being in the wrong directory. It will fail building if serial gc is not built in. I will separate these issues, this one for the serial gc gtest move, and another for the default gc. ------------- PR Comment: https://git.openjdk.org/jdk/pull/31049#issuecomment-4406283644
