On Mon, 20 Mar 2023 20:53:41 GMT, Eirik Bjorsnos <d...@openjdk.org> wrote:
> Please review this PR which removes the following outdated guard/check from > SunCommandLineLauncher: > > > if ((options.indexOf("-Djava.compiler=") != -1) && > (options.toLowerCase().indexOf("-djava.compiler=none") == -1)) { > throw new IllegalConnectorArgumentsException("Cannot debug with a JIT > compiler", > ARG_OPTIONS); > } > > > > Efforts are underway to remove the `java.compiler` system property entirely, > besides that this test no longer makes sense since debugging with a JIT has > been supported for a while. Looks good. This was actually found years ago but the issue closed: [JDK-6374661](https://bugs.openjdk.org/browse/JDK-6374661) ------------- Marked as reviewed by dholmes (Reviewer). PR Review: https://git.openjdk.org/jdk/pull/13109#pullrequestreview-1349624636