> After the integration of #2565 three subtests of 
> `compiler/valhalla/inlinetypes/TestValueConstruction.java` started failing, 
> but only when `-XX:+StressUnstableIfTraps` was passed. This PR fixes this by 
> introducing new subtasks with a higher memlimit for the affected subtests.
> 
> Alternatively, I could have simply raised the memlimit, but this would 
> diminish the capability of these tests to detect regressions in the 
> scalarization strategy.
> 
> Testing:
>  - [x] 50 repetitions of 
> `compiler/valhalla/inlinetypes/TestValueConstruction.java`
>  - [x] 50 repetitions of 
> `compiler/valhalla/inlinetypes/TestValueConstruction.java` with 
> `-XX:+StressUnstableIfTraps`
> 
> 
> ---------
> - [x] I confirm that I make this contribution in accordance with the [OpenJDK 
> Interim AI Policy](https://openjdk.org/legal/ai).

Manuel Hässig has updated the pull request incrementally with one additional 
commit since the last revision:

  It was fun while it lasted

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

Changes:
  - all: https://git.openjdk.org/valhalla/pull/2599/files
  - new: https://git.openjdk.org/valhalla/pull/2599/files/db9d1d69..88b5659c

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=valhalla&pr=2599&range=01
 - incr: https://webrevs.openjdk.org/?repo=valhalla&pr=2599&range=00-01

  Stats: 45 lines in 1 file changed: 0 ins; 45 del; 0 mod
  Patch: https://git.openjdk.org/valhalla/pull/2599.diff
  Fetch: git fetch https://git.openjdk.org/valhalla.git pull/2599/head:pull/2599

PR: https://git.openjdk.org/valhalla/pull/2599

Reply via email to