> 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: * extract g1 as default changes ------------- Changes: - all: https://git.openjdk.org/jdk/pull/31049/files - new: https://git.openjdk.org/jdk/pull/31049/files/4a3a3c12..b278aac8 Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=31049&range=02 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=31049&range=01-02 Stats: 7 lines in 2 files changed: 4 ins; 0 del; 3 mod Patch: https://git.openjdk.org/jdk/pull/31049.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/31049/head:pull/31049 PR: https://git.openjdk.org/jdk/pull/31049
