RFR: 8288781: C1: LIR_OpVisitState::maxNumberOfOperands too small

Martin Doerr mdoerr at openjdk.org
Tue Jun 21 21:25:19 UTC 2022


On Tue, 21 Jun 2022 09:13:51 GMT, Richard Reingruber <rrich at openjdk.org> wrote:

> Increment `LIR_OpVisitState::maxNumberOfOperands` by 1 to allow C1 compilation of a method that receives 21 parameters in registers instead of crashing.
> 
> Add regression test. The regression test crashes on ppc because there all parameters (8 integer + 13 float = 21) can be passed in registers.
> 
> The fix passed our CI testing. This includes most JCK and JTREG test, also in Xcomp mode, on the standard platforms and also on Linux/PPC64le.

Thanks for fixing it and providing a test. LGTM.

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

Marked as reviewed by mdoerr (Reviewer).

PR: https://git.openjdk.org/jdk19/pull/51


More information about the hotspot-compiler-dev mailing list