Integrated: 8262508: Vector API's ergonomics is incorrect

[Jie Fu]

On Mon, 1 Mar 2021 02:11:50 GMT, Jie Fu <jiefu at openjdk.org> wrote:

> Hi all,
> 
> The ergonomics (e.g., EnableVectorReboxing/EnableVectorAggressiveReboxing) for Vector API doesn't work as expected.
> 
> There are two sets of rules for Vector API.
> One is in Arguments::apply_ergo() [1], which sets the flags to be false.
> And the other is in Modules::define_module() [2], which sets the flags to be true.
> 
> The value of EnableVectorReboxing and EnableVectorAggressiveReboxing may be changed (from specified true to false) in [1] if EnableVectorSupport=false (which is the default value).
> It would be wrong when EnableVectorSupport isn't specified in the command line since it will be set true later in [2]
> So rules in [1] should not be executed if EnableVectorSupport isn't specified.
> 
> Thanks.
> Best regards,
> Jie
> 
> [1] https://github.com/openjdk/jdk/blob/master/src/hotspot/share/runtime/arguments.cpp#L4081
> [2] https://github.com/openjdk/jdk/blob/master/src/hotspot/share/classfile/modules.cpp#L462

This pull request has now been integrated.

Changeset: 351889f1
Author:    Jie Fu <jiefu at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/351889f1
Stats:     83 lines in 2 files changed: 82 ins; 0 del; 1 mod

8262508: Vector API's ergonomics is incorrect

Reviewed-by: vlivanov

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

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