RFR: 8252505: C1/C2 compiler support for blackholes [v11]

Aleksey Shipilev shade at openjdk.java.net
Tue Dec 1 15:29:57 UTC 2020


On Tue, 1 Dec 2020 12:30:12 GMT, Vladimir Ivanov <vlivanov at openjdk.org> wrote:

>> Aleksey Shipilev has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Cleanup BlackholeCallGenerator
>
> src/hotspot/share/c1/c1_GraphBuilder.cpp line 2032:
> 
>> 2030:   if (target->is_loaded() && target->return_type()->basic_type() == T_VOID &&
>> 2031:       compilation()->directive()->should_blackhole(target)) {
>> 2032:     if (try_blackhole(target)) {
> 
> Since `try_blackhole` can't fail, why do the check at all?

Right. This used to be modeled after `try_inline`, which could bail. `try_blackhole` cannot bail, so there is no reason to assert it. Fixed.

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

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


More information about the hotspot-dev mailing list