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

Tobias Hartmann thartmann at openjdk.org
Wed Jun 22 05:57:01 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.

Looks good.

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

Marked as reviewed by thartmann (Reviewer).

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


More information about the hotspot-compiler-dev mailing list