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