RFR: 8275104: IR framework does not handle client VM builds correctly [v2]

Christian Hagedorn chagedorn at openjdk.java.net
Thu Oct 21 07:04:29 UTC 2021


> While the IR framework is primarily used for C2 IR verification, it should also work with client VM builds. There are currently some problems which are fixed with this patch:
> 
> - The IR framework currently only bails out of IR matching if C2 is excluded by command line flags. However, when running an IR JTreg test with a client VM build, IR matching fails when not specifically adding `@requires vm.compiler2.enabled` to exclude the test.
> - `@Test` and `@ForceCompile` do not work correctly and throw an exception due to an incompatible compilation level selection without C2.
> - Some internal framework tests fail (the fix also improves `TestDIgnoreCompilerControls` in general).
> 
> Testing:
> 
> - Standard tier testing
> - Testing internal framework tests with standard build (tiered), client VM (without C2) and server VM build (without C1)
> 
> Thanks,
> Christian

Christian Hagedorn has updated the pull request incrementally with one additional commit since the last revision:

  Change MinInliningThreshold to TypeProfileLevel in IRExample

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

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/6037/files
  - new: https://git.openjdk.java.net/jdk/pull/6037/files/7e256e6d..1670bcea

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=6037&range=01
 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=6037&range=00-01

  Stats: 2 lines in 1 file changed: 0 ins; 0 del; 2 mod
  Patch: https://git.openjdk.java.net/jdk/pull/6037.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/6037/head:pull/6037

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


More information about the hotspot-compiler-dev mailing list