RFR: 8264957: Cleanup unused array Type::dual_type [v2]

Tobias Hartmann thartmann at openjdk.java.net
Tue Apr 13 07:39:59 UTC 2021


On Tue, 13 Apr 2021 06:39:18 GMT, Xiaohong Gong <xgong at openjdk.org> wrote:

>> This is a bug fix for [1] which adds a new vector mask type. The new added TYPE `"VectorMask"` is inserted into `enum TYPES`, while the array `"Type::dual_type"` is not updated. This makes the array elements are not aligned with TYPES.
>> 
>> I met the following crash due to this issue when I was working on the masking feature support on panama-vector:
>> 
>>  Internal Error (/home/xiagon01/code/panama-vector/src/hotspot/share/opto/type.hpp:1727), pid=104432, tid=104449
>>  # assert(_base >= AnyPtr && _base <= KlassPtr) failed: Not a pointer
>> 
>> Adding a value like other vector types for the `"VectorMask"` in the array `"dual_type"` can fix it.
>> 
>> [1] https://bugs.openjdk.java.net/browse/JDK-8262355
>> 
>> Tested with tier1 and jdk:tier3
>
> Xiaohong Gong has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Remove array Type::dual_type

Looks good to me.

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

Marked as reviewed by thartmann (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/3410


More information about the hotspot-compiler-dev mailing list