RFR: 8266251: compiler.inlining.InlineAccessors shouldn't run a test in driver VM
Igor Ignatyev
iignatyev at openjdk.java.net
Wed Apr 28 21:10:17 UTC 2021
Hi all,
could you please review the patch which removes redundant and "incorrect" testing from `InlineAccessors` test? as `InlineAccessors::doTest` is called by the JVM which runs `Launcher` class, we aren't actually losing any coverage.
from JBS:
> `compiler.inlining.InlineAccessors` test executes `doTest` method in its `InlineAccessors::main`, but InlineAccessors is run in driver mode (so it's not expected to do actual testing and strictly speaking can be executed in JVM other than JVM under test).
Thanks,
-- Igor
-------------
Commit messages:
- 8266251
Changes: https://git.openjdk.java.net/jdk/pull/3774/files
Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=3774&range=00
Issue: https://bugs.openjdk.java.net/browse/JDK-8266251
Stats: 6 lines in 1 file changed: 0 ins; 4 del; 2 mod
Patch: https://git.openjdk.java.net/jdk/pull/3774.diff
Fetch: git fetch https://git.openjdk.java.net/jdk pull/3774/head:pull/3774
PR: https://git.openjdk.java.net/jdk/pull/3774
More information about the hotspot-compiler-dev
mailing list