On Fri, 8 May 2026 12:27:33 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:
> 
>   * extract g1 as default changes

All right, that makes sense. The synopsis needs to be changed to reflect the 
new version of this PR.

-------------

PR Comment: https://git.openjdk.org/jdk/pull/31049#issuecomment-4407066077

Reply via email to