On Thu, 5 May 2022 19:27:47 GMT, Paul Sandoz <[email protected]> wrote:
>> Xiaohong Gong has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Rename "use_predicate" to "needs_predicate"
>
> src/hotspot/share/opto/vectorIntrinsics.cpp line 1238:
>
>> 1236: } else {
>> 1237: // Masked vector load with IOOBE always uses the predicated load.
>> 1238: const TypeInt* offset_in_range =
>> gvn().type(argument(8))->isa_int();
>
> Should it be `argument(7)`? (and adjustments later to access the container).
I'm afraid it's `argument(8)` for the load operation since the `argument(7)` is
the mask input. It seems the argument number is not right begin from the mask
input which is expected to be `6`. But the it's not. Actually I don't quite
understand why.
-------------
PR: https://git.openjdk.java.net/jdk/pull/8035
