RFR: 8325438: Add exhaustive tests for Math.round intrinsics [v6]
Hamlin Li
mli at openjdk.org
Thu Mar 14 11:23:56 UTC 2024
> HI,
> Can you have a look at this patch adding some tests for Math.round instrinsics?
> Thanks!
>
> ### FYI:
> During the development of RoundVF/RoundF, we faced the issues which were only spotted by running test exhaustively against 32/64 bits range of int/long.
> It's helpful to add these exhaustive tests in jdk for future possible usage, rather than build it everytime when needed.
> Of course, we need to put it in `manual` mode, so it's not run when `-automatic` jtreg option is specified which I guess is the mode CI used, please correct me if I'm assume incorrectly.
Hamlin Li 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 ten additional commits since the last revision:
- Fix test failure in TestRoundVectorDoubleRandom.java
- Merge branch 'master' into round-v-exhaustive-tests
- add comments; refine code
- refine code; fix bug
- Merge branch 'master' into round-v-exhaustive-tests
- fix issue
- mv tests
- use IR framework to construct the random tests
- Initial commit
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/17753/files
- new: https://git.openjdk.org/jdk/pull/17753/files/2afa8160..3f50c062
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=17753&range=05
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=17753&range=04-05
Stats: 98690 lines in 2066 files changed: 16605 ins; 76003 del; 6082 mod
Patch: https://git.openjdk.org/jdk/pull/17753.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/17753/head:pull/17753
PR: https://git.openjdk.org/jdk/pull/17753
More information about the hotspot-compiler-dev
mailing list