On Wed, 17 Jun 2026 07:24:30 GMT, Marc Chevalier <[email protected]> wrote:

> We need to take the `HALF` parameters for long/double, in the same way as we 
> do for `CallGenerator::do_late_inline_helper()` or 
> `GraphKit::set_arguments_for_java_call`. Added a targeted assert.
> 
> Thanks,
> Marc
> 
> ---------
> - [x] I confirm that I make this contribution in accordance with the [OpenJDK 
> Interim AI Policy](https://openjdk.org/legal/ai).

src/hotspot/share/opto/vector.cpp line 208:

> 206:       kit.ensure_stack(kit.sp() + nargs);
> 207:       uint in_idx = TypeFunc::Parms;
> 208:       int parm_idx = 0;

It is useful to explain that `i` is the index of the argument on the JVM stack, 
while `parm_idx` is the index of the argument in reality,

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

PR Review Comment: 
https://git.openjdk.org/valhalla/pull/2561#discussion_r3427019997

Reply via email to