Integrated: 8279282: [vectorapi] Matcher::supports_vector_comparison_unsigned is not needed on x86
Quan Anh Mai
duke at openjdk.java.net
Thu Jan 20 18:28:52 UTC 2022
On Wed, 5 Jan 2022 10:30:46 GMT, Quan Anh Mai <duke at openjdk.java.net> wrote:
> Hi,
>
> Currently, unsigned comparison on AVX is implemented by zero extending elements and comparing the results. This leads to unnecessary complexity. This patch changes the implementation to use the identity existing in `Integer/Long.compareUnsigned`, that is `compareUnsigned(x, y) == compare(x ^ min_value, y ^ min_value)`.
>
> Thank you very much.
This pull request has now been integrated.
Changeset: 02390c79
Author: Quan Anh Mai <anhmdq99 at gmail.com>
Committer: Sandhya Viswanathan <sviswanathan at openjdk.org>
URL: https://git.openjdk.java.net/jdk/commit/02390c79b1acff1a953d29c6f70623f3b7838698
Stats: 195 lines in 8 files changed: 65 ins; 99 del; 31 mod
8279282: [vectorapi] Matcher::supports_vector_comparison_unsigned is not needed on x86
Reviewed-by: kvn, sviswanathan, vlivanov
-------------
PR: https://git.openjdk.java.net/jdk/pull/6966
More information about the hotspot-compiler-dev
mailing list