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