RFR: 8322959: vectorapi: get wrong argument for `limit` in indexPartiallyInUpperRange intrinsic

Sandhya Viswanathan sviswanathan at openjdk.org
Wed Jan 3 17:12:49 UTC 2024


On Wed, 3 Jan 2024 16:05:48 GMT, Hamlin Li <mli at openjdk.org> wrote:

> Hi,
> Can you review this simple fix for indexPartiallyInUpperRange intrinsic?
> Thanks.

src/hotspot/share/opto/vectorIntrinsics.cpp line 3151:

> 3149: 
> 3150:   Node* offset = argument(3);
> 3151:   Node* limit = argument(4);

The offset is of long type so will take 2 spots (3 and 4) of argument. So limit will be argument(5).  The original code (limit = argument(5)) looks correct to me.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/17247#discussion_r1440691288


More information about the hotspot-compiler-dev mailing list