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