Skip to content

Commit eaa6a82

Browse files
author
Jatin Bhateja
committed
Minor cleanups
1 parent 7044950 commit eaa6a82

File tree

6 files changed

+4
-18
lines changed

6 files changed

+4
-18
lines changed

src/hotspot/cpu/aarch64/vm_version_aarch64.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -702,7 +702,7 @@ void VM_Version::print_platform_virtualization_info(outputStream* st) {
702702
#endif
703703
}
704704

705-
void VM_Version::pre_initialize(void) {}
705+
void VM_Version::pre_initialize() {}
706706

707707
void VM_Version::initialize_cpu_information(void) {
708708
// do nothing if cpu info has been initialized

src/hotspot/cpu/arm/vm_version_arm_32.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -353,7 +353,7 @@ void VM_Version::initialize() {
353353
_is_initialized = true;
354354
}
355355

356-
void VM_Version::pre_initialize(void) {}
356+
void VM_Version::pre_initialize() {}
357357

358358
void VM_Version::initialize_cpu_information(void) {
359359
// do nothing if cpu info has been initialized

src/hotspot/share/runtime/abstract_vm_version.hpp

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -57,8 +57,6 @@ class Abstract_VM_Version: AllStatic {
5757
// CPU feature flags, can be affected by VM settings.
5858
static uint64_t _features;
5959

60-
// Extra CPU feature flags used when all 64 bits of _features are exhausted for
61-
// on a given target, currently only used for x86_64, can be affected by VM settings.
6260
static const char* _features_string;
6361

6462
// Original CPU feature flags, not affected by VM settings.

src/jdk.internal.vm.ci/share/classes/jdk/vm/ci/hotspot/HotSpotJVMCIBackendFactory.java

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,6 @@ static <CPUFeatureType extends Enum<CPUFeatureType>> EnumSet<CPUFeatureType> con
5858
Class<CPUFeatureType> enumType,
5959
Map<String, Long> constants,
6060
long features,
61-
long extra_features,
6261
Map<String, String> renaming) {
6362
EnumSet<CPUFeatureType> outFeatures = EnumSet.noneOf(enumType);
6463
List<String> missing = new ArrayList<>();
@@ -76,17 +75,6 @@ static <CPUFeatureType extends Enum<CPUFeatureType>> EnumSet<CPUFeatureType> con
7675
missing.add(name);
7776
}
7877
}
79-
if (key.startsWith("VM_Version::EXTRA_CPU_")) {
80-
String name = key.substring("VM_Version::EXTRA_CPU_".length());
81-
try {
82-
CPUFeatureType feature = Enum.valueOf(enumType, renaming.getOrDefault(name, name));
83-
if ((extra_features & bitMask) != 0) {
84-
outFeatures.add(feature);
85-
}
86-
} catch (IllegalArgumentException iae) {
87-
missing.add(name);
88-
}
89-
}
9078
}
9179
if (!missing.isEmpty()) {
9280
throw new JVMCIError("Missing CPU feature constants: %s", missing);

src/jdk.internal.vm.ci/share/classes/jdk/vm/ci/hotspot/aarch64/AArch64HotSpotJVMCIBackendFactory.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ public class AArch64HotSpotJVMCIBackendFactory implements HotSpotJVMCIBackendFac
4949
private static EnumSet<AArch64.CPUFeature> computeFeatures(AArch64HotSpotVMConfig config) {
5050
// Configure the feature set using the HotSpot flag settings.
5151
Map<String, Long> constants = config.getStore().getConstants();
52-
return HotSpotJVMCIBackendFactory.convertFeatures(CPUFeature.class, constants, config.vmVersionFeatures, 0L, emptyMap());
52+
return HotSpotJVMCIBackendFactory.convertFeatures(CPUFeature.class, constants, config.vmVersionFeatures, emptyMap());
5353
}
5454

5555
private static EnumSet<AArch64.Flag> computeFlags(AArch64HotSpotVMConfig config) {

src/jdk.internal.vm.ci/share/classes/jdk/vm/ci/hotspot/riscv64/RISCV64HotSpotJVMCIBackendFactory.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ public class RISCV64HotSpotJVMCIBackendFactory implements HotSpotJVMCIBackendFac
4949
private static EnumSet<RISCV64.CPUFeature> computeFeatures(RISCV64HotSpotVMConfig config) {
5050
// Configure the feature set using the HotSpot flag settings.
5151
Map<String, Long> constants = config.getStore().getConstants();
52-
return HotSpotJVMCIBackendFactory.convertFeatures(CPUFeature.class, constants, config.vmVersionFeatures, 0L, emptyMap());
52+
return HotSpotJVMCIBackendFactory.convertFeatures(CPUFeature.class, constants, config.vmVersionFeatures, emptyMap());
5353
}
5454

5555
private static EnumSet<RISCV64.Flag> computeFlags(RISCV64HotSpotVMConfig config) {

0 commit comments

Comments
 (0)