[lworld+vector] RFR: 8311675: [lworld+vector] Max Species support. [v4]

[Xiaohong Gong]

On Mon, 16 Oct 2023 18:25:00 GMT, Jatin Bhateja <jbhateja at openjdk.org> wrote:

>> - Patch adds MaxSpecies support for all types of vectors.
>> - New factory methods and VectorPayload classes for various kinds of vector, shuffle and mask payloads.
>> - Disable CDS archiving of VectorPayload classes.
>> - Summary of high level flow :-
>>   1/ Max species payloads encapsulate @multifield annotated field accepting -1 value as bundle size parameter.
>>   2/ For Vector payload bundle size is determined using maximum vector size supported by the target. 
>>   3/ For Shuffles and Masks payloads multifield bundle size is a function of maximum vector size and vector lane size.
>>   4/ Based on the dynamic bundle size parser creates a separate FieldInfo structure for each base and synthetic multifield and rest of the flow remains the same. 
>> 
>> Kindly review and share your feedback.
>> 
>> Best Regards,
>> Jatin
>
> Jatin Bhateja has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Review comments resolutions.

src/hotspot/share/ci/ciEnv.cpp line 1776:

> 1774: bool ciEnv::is_multifield_scalarized(BasicType bt, int vec_length) {
> 1775: #if COMPILER2
> 1776:   CompilerThread* ct = CompilerThread::current();

Could you please change `#if COMPILER2` to `#ifdef COMPILER2` ? This makes the jdk build error with `--configure-args --with-jvm-features=-compiler2`

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

PR Review Comment: https://git.openjdk.org/valhalla/pull/931#discussion_r1361432201