RFR: 8238737: remove DeoptimizeAllClassesRate from CTW library

Igor Ignatyev iignatyev at openjdk.java.net
Wed Sep 30 14:53:22 UTC 2020


On Wed, 30 Sep 2020 09:32:47 GMT, Vladimir Ivanov <vlivanov 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.

Aleksey, Vladimir, thank you for your reviews.

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

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


More information about the hotspot-compiler-dev mailing list