8390807: Reduce run time of test AOTCodeFlags.java - #32473
Conversation
|
👋 Welcome back iklam! A progress list of the required criteria for merging this PR into |
|
❗ This change is not yet ready to be integrated. |
|
The total number of required reviews for this PR has been set to 2 based on the presence of this label: |
Webrevs
|
vnkozlov
left a comment
There was a problem hiding this comment.
Good. Thank you for doing it.
|
In #30660 (comment) , @vnkozlov tested with the "HelloWorldApp" and saw failures when running with With this PR, I reran the same tiers and all tests passed.So it looks like it's safe to use "AOTCodeSimpleTestApp". |
This test runs JavacBench 16 times so it takes long time to finish.
This PR tries to replace the test app with a smaller program, but run with
-Xbatchand use a large enough number of iterations to ensure that C1/C2 stubs are generated.Progress
Issue
Reviewers
Reviewing
Using
gitCheckout this PR locally:
$ git fetch https://git.openjdk.org/jdk.git pull/32473/head:pull/32473$ git checkout pull/32473Update a local copy of the PR:
$ git checkout pull/32473$ git pull https://git.openjdk.org/jdk.git pull/32473/headUsing Skara CLI tools
Checkout this PR locally:
$ git pr checkout 32473View PR using the GUI difftool:
$ git pr show -t 32473Using diff file
Download this PR as a diff file:
https://git.openjdk.org/jdk/pull/32473.diff
Using Webrev
Link to Webrev Comment