RFR: 8286104: use aggressive liveness for unstable_if traps [v7]

Xin Liu xliu at openjdk.java.net
Wed Jun 1 20:35:37 UTC 2022


On Wed, 1 Jun 2022 19:58:51 GMT, Vladimir Kozlov <kvn at openjdk.org> wrote:

>> Xin Liu has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Remove useless flag. if jdwp is on, liveness_at_bci() marks all local
>>   variables live.
>
> src/hotspot/share/opto/compile.cpp line 1877:
> 
>> 1875: uint unstable_ifs_all              = 0;
>> 1876: 
>> 1877: void Compile::preprocess_unstable_ifs() {
> 
> I assume it will be used in product VM  by next changes 8287385.
> Otherwise counters and method should be under `#ifndef PRODUCT` because they used only for statistics.

Yes, this is for 8287385. I will remove preprocessing logic from this PR. 

Because we can determine a unstable_if trap is trivial after parsing.  My idea is to do the leftover parsing job in this preprocess function. That's why I think preprocess should before `PhaseRemoveUseless`.

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

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


More information about the hotspot-compiler-dev mailing list