RFR: 8373654: Tests in sources/ should only run once [v3]
David Holmes
dholmes at openjdk.org
Tue Dec 16 11:53:11 UTC 2025
> This test is checking for backsliding on the use of NULL in the hotspot source files. The test only needs to be run once per change but it is defined as a regular tier1 test which then gets run for each os-arch combination, plus product and debug, and then again with a bunch of flags in tier3. This is just a waste of resources. At a minimum we should make the test flagless so that it doesn't run in tier3, but we can also restrict it to a single platform.
>
> Before the change the test ran 27 times in tier1-3 in our CI. After it runs once.
>
>
> EDIT: Updated to include both tests in sources/
>
> Thanks
David Holmes has updated the pull request incrementally with one additional commit since the last revision:
reviewer feedback
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/28818/files
- new: https://git.openjdk.org/jdk/pull/28818/files/d06591b5..8fb98814
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=28818&range=02
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=28818&range=01-02
Stats: 2 lines in 2 files changed: 0 ins; 0 del; 2 mod
Patch: https://git.openjdk.org/jdk/pull/28818.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/28818/head:pull/28818
PR: https://git.openjdk.org/jdk/pull/28818
More information about the hotspot-dev
mailing list