RFR: 8307625: Redundant receiver null check in LibraryCallKit::generate_method_call [v3]

[Christian Hagedorn]

On Mon, 26 Jun 2023 13:05:13 GMT, Eric Nothum <duke at openjdk.org> wrote:

>> The null_check_receiver() calls in generate_method_call are redundant as all callers of generate_method_call already perform this check. For future uses of generate_method_call a new assert is introduced that fails if the caller does not null check the receiver.
>> Also generally null_check_receiver() should be combined with stopped(), which was not the case here.
>
> Eric Nothum has updated the pull request incrementally with one additional commit since the last revision:
> 
>   8307625: adding null_check_receiver() for the uninitialized case, as else only argument(1) is null checked

Thanks for adding the null check as discussed offline together with @TobiHartmann. Looks good!

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

Marked as reviewed by chagedorn (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/14542#pullrequestreview-1498475996