Integrated: 8292511: AArch64: Align CPU feature name for NEON with hwcap
Pengfei Li
pli at openjdk.org
Fri Aug 19 09:11:56 UTC 2022
On Thu, 18 Aug 2022 08:13:29 GMT, Pengfei Li <pli at openjdk.org> wrote:
> AArch64 hwcap uses `asimd` to represent the CPU feature of NEON. But in
> vm_version_aarch64.hpp, this name is redefined as `simd`. This renaming
> looks trivial but can easily lead to bugs in test cases as more and more
> people start writing jtreg IR tests today.
>
> For example, if someone just finds CPU feature name from some documents
> or `/proc/cpuinfo`, and writes below annotations in his test case, the
> test case won't run on AArch64.
> - `@requires vm.cpu.features ~= ".*asimd.*"`
> - `@IR(applyIfCPUFeature = {"asimd", "true"}, counts = {...})`
>
> So in this patch we propose to align the CPU feature names. Some related
> jtreg cases are also updated. There's another misalignment of the names
> for CPU feature `sha2`. We don't change it as we have some discussions
> in https://bugs.openjdk.org/browse/JDK-8285266.
>
> Tested tier1 and hotspot:compiler/vectorapi on x86 and AArch64.
This pull request has now been integrated.
Changeset: f2f0cd86
Author: Pengfei Li <pli at openjdk.org>
URL: https://git.openjdk.org/jdk/commit/f2f0cd86bf4dce4633f484476077fd090549780e
Stats: 17 lines in 5 files changed: 0 ins; 0 del; 17 mod
8292511: AArch64: Align CPU feature name for NEON with hwcap
Reviewed-by: aph, njian
-------------
PR: https://git.openjdk.org/jdk/pull/9913
More information about the hotspot-dev
mailing list