RFR: 8292511: AArch64: Align CPU feature name for NEON with hwcap
    Pengfei Li 
    pli at openjdk.org
       
    Thu Aug 18 08:21:43 UTC 2022
    
    
  
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.
-------------
Commit messages:
 - 8292511: AArch64: Align CPU feature name for NEON with hwcap
Changes: https://git.openjdk.org/jdk/pull/9913/files
 Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=9913&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8292511
  Stats: 17 lines in 5 files changed: 0 ins; 0 del; 17 mod
  Patch: https://git.openjdk.org/jdk/pull/9913.diff
  Fetch: git fetch https://git.openjdk.org/jdk pull/9913/head:pull/9913
PR: https://git.openjdk.org/jdk/pull/9913
    
    
More information about the hotspot-dev
mailing list