JDK-8302135 : Remove try/catch from tests where it might hide failures
Type:Enhancement
Component:hotspot
Sub-Component:compiler
Affected Version:21
Priority:P4
Status:Open
Resolution:Unresolved
Submitted:2023-02-09
Updated:2023-03-23
The Version table provides details related to the release that this issue/RFE will be addressed.
Unresolved : Release in which this issue/RFE will be addressed. Resolved: Release in which this issue/RFE has been resolved. Fixed : Release in which this issue/RFE has been fixed. The release containing this fix may be available for download as an Early Access Release or a General Availability Release.
As discussed in PR
https://github.com/openjdk/jdk/pull/12475
Comments
On a first search through all tests in
test/hotspot/jtreg/compiler
I found the following tests to be candidates for further investigation.
It may well be that we will have to accept many of them as they are, because we do not want to risk breaking the test. Modifying some try/catch statements can lead to tests not reproducing in some cases.
test/hotspot/jtreg/compiler/arraycopy/TestArrayCopyWithBadOffset.java
test/hotspot/jtreg/compiler/arraycopy/TestNegativeArrayCopyAfterLoop.java
test/hotspot/jtreg/compiler/c1/Test7123108.java
test/hotspot/jtreg/compiler/c1/Test8271202.java
test/hotspot/jtreg/compiler/c1/Test8275337.java
test/hotspot/jtreg/compiler/c1/TestValueNumberingNullObject.java
test/hotspot/jtreg/compiler/c2/cr6646020/Tester.java
test/hotspot/jtreg/compiler/c2/cr6865031/Test.java
test/hotspot/jtreg/compiler/c2/irTests/TestDebugInfo.java
test/hotspot/jtreg/compiler/c2/Test5057225.java
test/hotspot/jtreg/compiler/c2/Test6603011.java
test/hotspot/jtreg/compiler/c2/Test6646019.java
test/hotspot/jtreg/compiler/c2/Test6799693.java
test/hotspot/jtreg/compiler/c2/Test6800154.java
test/hotspot/jtreg/compiler/c2/Test6805724.java
test/hotspot/jtreg/compiler/c2/Test6837094.java
test/hotspot/jtreg/compiler/c2/Test6880034.java
test/hotspot/jtreg/compiler/c2/Test7125879.java
test/hotspot/jtreg/compiler/c2/TestDuplicateSimpleLoopBackedge.java
test/hotspot/jtreg/compiler/codegen/aes/TestCipherBlockChainingEncrypt.java
test/hotspot/jtreg/compiler/codegen/Test6431242.java
test/hotspot/jtreg/compiler/codegen/Test6823354.java
test/hotspot/jtreg/compiler/escapeAnalysis/cr6716441/Tester.java
test/hotspot/jtreg/compiler/escapeAnalysis/Test6896727.java
test/hotspot/jtreg/compiler/escapeAnalysis/TestEliminateLocksOffCrash.java - ok?
test/hotspot/jtreg/compiler/escapeAnalysis/TestIdealAllocShape.java - ok?
test/hotspot/jtreg/compiler/exceptions/TestRecursiveReplacedException.java
test/hotspot/jtreg/compiler/gcbarriers/UnsafeIntrinsicsTest.java
test/hotspot/jtreg/compiler/intrinsics/object/TestClone.java
test/hotspot/jtreg/compiler/intrinsics/string/TestStringConstruction.java
test/hotspot/jtreg/compiler/jsr292/InvokerSignatureMismatch.java
test/hotspot/jtreg/compiler/jsr292/Test7082949.java
test/hotspot/jtreg/compiler/jvmci/events/JvmciShutdownEventListener.java
test/hotspot/jtreg/compiler/loopopts/LoopRotateBadNodeBudget.java
test/hotspot/jtreg/compiler/loopopts/TestCreateNewIfForPredicateCloning.java
test/hotspot/jtreg/compiler/loopopts/TestLoopPeeling.java
test/hotspot/jtreg/compiler/loopopts/TestPeelingSkeletonPredicateInitialization.java
test/hotspot/jtreg/compiler/profiling/spectrapredefineclass/Agent.java
test/hotspot/jtreg/compiler/profiling/spectrapredefineclass_classloaders/Agent.java
test/hotspot/jtreg/compiler/profiling/TestUnexpectedProfilingMismatch.java
test/hotspot/jtreg/compiler/rangechecks/TestRangeCheckCmpUUnderflow.java