Integrated: 8266184: a few compiler/debug tests don't check exit code

[Igor Ignatyev]

On Wed, 28 Apr 2021 05:06:19 GMT, Igor Ignatyev <iignatyev at openjdk.org> wrote:

> Hi all,
> 
> could you please review this small fix for `compiler/debug` tests?
> from JBS:
>> `compiler/debug/TestGenerateStressSeed.java`, `TestStressCM.java`, and `TestStressIGVNAndCCP.java` don't check the exit code of spawned JVMs, as a result, they might miss misconfiguration and/or defects. in fact, `TestStressIGVNAndCCP.java` uses a wrong flag name (`TraceIterativeIGVN` instead of `TraceIterativeGVN`) and this was unnoticed b/c we didn't check the exit code.
> 
> the patch adds calls to `OutputAnalyzer::shouldHaveExitValue` and fixes `TestStressIGVNAndCCP.java` to use the right flag.
> 
> testing:
> - [x] `test/hotspot/jtreg/compiler/debug` on `linux-x64-debug`
> 
> Thanks,
> -- Igor

This pull request has now been integrated.

Changeset: 19d3c451
Author:    Igor Ignatyev <iignatyev at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/19d3c451b89680cefa41fd18f9e9ba0624f56858
Stats:     7 lines in 3 files changed: 3 ins; 1 del; 3 mod

8266184: a few compiler/debug tests don't check exit code

Reviewed-by: thartmann

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

PR: https://git.openjdk.java.net/jdk/pull/3748