RFR: 8302149: Speed up compiler/jsr292/methodHandleExceptions/TestAMEnotNPE.java

Emanuel Peter epeter at openjdk.org
Fri Feb 17 10:30:53 UTC 2023


On Thu, 16 Feb 2023 18:25:44 GMT, Vladimir Kozlov <kvn at openjdk.org> wrote:

>> It takes just a tiny bit longer. `2.3` seconds for that run, whereas with `-XX:CompileCommand=compileonly,*::test` I get `2` seconds.
>
> I suggest then to use my proposal - compile all test's methods. I concern that compiling only one method may not catch some issues when the method is inlined.

@vnkozlov Actually, I just checked what methods it wants to compile. And the generated `test` methods are these:

p.Tdirect::test
p.T::test
p.Treflect::test


There is also a method `p.I::m` that seems to have been generated. Does that also need to be compiled?
I also see a `p.C::<init>` and `p.E::<init>.` And some class or interface `p.D`. And `p.F::<init>` and `p.F::m`.

@iwanowww do you think we need to include all of these to be compiled?

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

PR: https://git.openjdk.org/jdk/pull/12536


More information about the hotspot-compiler-dev mailing list