RFR: 8271461: CompileCommand support for hidden class methods [v4]
Ioi Lam
iklam at openjdk.java.net
Wed Aug 4 03:03:30 UTC 2021
On Wed, 4 Aug 2021 00:11:24 GMT, Jie Fu <jiefu at openjdk.org> wrote:
> > It still appears to apply to package names, not just class names.
>
> `is_hidden_calss_pattern` will check whether '*' is followed by "::" so cases like `java.util/*.ResourceBundle$$Lambda$1/0x<digits>::method` won't be detected as hidden class pattern. @dean-long
You also need this test case: `java.util/*::method`. It should be disallowed. Otherwise it will be transformed to `java.util+*::method`. (But I think you current code allows it).
I have the feeling that we are unnecessarily complicating the VM simply because PrintCompilation prints out `/`.
Why don't we change PrintCompilation to print out `+` instead?
It's not like anyone is depending of the current output of `/`, because it isn't accepted by `-XX:CompileCommand` anyway.
-------------
PR: https://git.openjdk.java.net/jdk/pull/4926
More information about the hotspot-compiler-dev
mailing list