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

Xin Liu xliu at openjdk.org
Wed Jun 29 05:23:43 UTC 2022


On Tue, 28 Jun 2022 20:47:19 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 _path from UnstableIfTrap. remember _next_bci(int) is enough.
>
> src/hotspot/share/opto/c2_globals.hpp line 420:
> 
>> 418:                                                                             \
>> 419:   develop(bool, OptimizeUnstableIf, true,                                   \
>> 420:           "Optimize UnstableIf traps")                                      \
> 
> New name is good.
> 
> Why you changed it to `develop` flag which is available only in `debug` VM? I want to keep it `diagnostic` so we can switch it off in `product` VM too (with `-XX:+UnlockDiagnosticVMOptions` flag`)

I see. revert it.

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

PR: https://git.openjdk.org/jdk/pull/8545


More information about the hotspot-compiler-dev mailing list