RFR: 8238737: remove DeoptimizeAllClassesRate from CTW library

Vladimir Ivanov vlivanov at openjdk.java.net
Wed Sep 30 09:35:09 UTC 2020


On Tue, 29 Sep 2020 23:53:38 GMT, Igor Ignatyev <iignatyev at openjdk.org> wrote:

> Hi all,
> 
> after [8238247 ](https://bugs.openjdk.java.net/browse/JDK-8238247), `DeoptimizeAllClassesRate` property became
> completely useless, and as I mentioned in the
> [review](https://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2020-February/036927.html), I didn't (and still
> don't) have any idea why we added this at the first place.   I have looked into the old implementation of CTW (one in
> `/src/share/vm/classfile/classLoader.cpp`) both at the time Java impl. was added by
> [8012447](https://bugs.openjdk.java.net/browse/JDK-8012447) and the old impl. was removed by
> [8214917](https://bugs.openjdk.java.net/browse/JDK-8214917), and there is nothing that would suggest that we had/needed
> `DeoptimizeAllClassesRate`, it's not and has never been used by any of our tests, so I'm going to remove it.

Looks good.

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

Marked as reviewed by vlivanov (Reviewer).

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


More information about the hotspot-compiler-dev mailing list