[lworld] RFR: 8253724: [lworld] C1 compilation hits assert: Empty inline type access should be removed

Frederic Parain fparain at openjdk.java.net
Tue Sep 29 16:28:43 UTC 2020


On Tue, 29 Sep 2020 16:17:02 GMT, Tobias Hartmann <thartmann at openjdk.org> wrote:

>> Please review these changes fixing handling the handling of empty inline types in C1.
>> The changeset includes two new unit tests, but they are currently disabled because they are causing failures with C2
>> which has issues with empty inline types too (C2 will be fixed with a different patch).
>> Tested with Mach5, tiers 1 to 3.
>> 
>> Thank you,
>> 
>> Fred
>
> Looks good! I'll re-enable these tests once I've fixed C2.

Tobias,

Thank you for the review.

Fred

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

PR: https://git.openjdk.java.net/valhalla/pull/203


More information about the valhalla-dev mailing list