RFR: 8283667: [vectorapi] Vectorization for masked load with IOOBE with predicate feature [v3]
Xiaohong Gong
xgong at openjdk.java.net
Fri May 6 03:51:01 UTC 2022
On Thu, 5 May 2022 19:27:47 GMT, Paul Sandoz <psandoz at openjdk.org> 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
More information about the hotspot-compiler-dev
mailing list