RFR: 8330157: C2: Add a stress flag for bailouts [v6]

[Daniel Skantz]

On Mon, 1 Jul 2024 06:56:51 GMT, Emanuel Peter <epeter at openjdk.org> wrote:

>> Daniel Skantz has updated the pull request incrementally with two additional commits since the last revision:
>> 
>>  - tweak probability in CtwRunner, add CaptureBailoutInformation
>>  - remove once-used method arg
>
> test/hotspot/jtreg/compiler/debug/TestStressBailout.java line 39:
> 
>> 37:  * @summary Basic tests for bailout stress flag.
>> 38:  * @library /test/lib /
>> 39:  * @run driver compiler.debug.TestStressBailout
> 
> Are external flags passed through to your test-vm? Not sure what the impact of `driver` vs `main` is here, or if you would need to add the external flags explicitly when you create the child process.

External flags should propagate if we change `createLimitedTestJavaProcessBuilder` to `createTestJavaProcessBuilder`. We could make this change as the test should be stable under most other flags.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/19646#discussion_r1660881556