RFR: 8377438: [VectorAPI] Add new carrier type to LaneType class [v3]

Chen Liang liach at openjdk.org
Wed Feb 25 15:44:12 UTC 2026


On Wed, 25 Feb 2026 15:31:15 GMT, Jatin Bhateja <jbhateja at openjdk.org> wrote:

>> src/hotspot/share/opto/vectorIntrinsics.cpp line 46:
>> 
>>> 44: 
>>> 45:   ciField* fd1 = ik->get_field_by_name(ciSymbols::CTYPE_name(), ciSymbols::class_signature(), /* is_static */ true);
>>> 46:   assert(fd1 != nullptr, "element type info is missing");
>> 
>> We probably need separate asserts for CTYPE and ETYPE now, given these asserts test all fields. Below assert comment should get an update too.
>
> CTYPE is carrier type and it must be a java primitive type, In addition we have replaced ETYPE field with CTYPE from all concrete shuffle and mask classes. Thus CTYPE field primarily exists in all concrete mask / shuffle and vector classes and its type is being checked here.

Indeed you are right. My bad.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/29629#discussion_r2853810084


More information about the core-libs-dev mailing list