RFR: 8278947: Support for array constants in constant table [v2]

Vladimir Ivanov vlivanov at openjdk.java.net
Tue Feb 8 18:58:13 UTC 2022


On Tue, 4 Jan 2022 08:47:52 GMT, Quan Anh Mai <duke at openjdk.java.net> wrote:

>> Hi,
>> 
>> This patch adds support for arrays in compiled code constant tables and uses it for various vector replicate operations on x86.
>> 
>> Test: GHA, linux x64 tier 1-3
>> 
>> Thank you very much.
>
> Quan Anh Mai has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains 10 additional commits since the last revision:
> 
>  - Merge branch 'master' into constantVectorUpdate
>  - update copyright
>  - missing type in array_constant
>  - Merge branch 'master' into constantVector
>  - use constant table for remaining types
>  - Merge branch 'master' into constantVector
>  - refactor
>  - replicate using constant
>  - Merge branch 'master' into constantVector
>  - initial commit

Nice improvement!

src/hotspot/share/opto/constantTable.hpp line 41:

> 39:   private:
> 40:     BasicType _type;
> 41:     bool _is_array;

Can you rely on `_type` and just use `T_ARRAY` tag for that purpose?

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

Marked as reviewed by vlivanov (Reviewer).

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


More information about the hotspot-compiler-dev mailing list