8350663: AArch64: Enable UseSignumIntrinsic by default

cnqpzhang · adinn · commit b025d8c2e062 · 2025-03-18T15:44:28.000Z
Reviewed-by: adinn
diff --git a/src/hotspot/cpu/aarch64/vm_version_aarch64.cpp b/src/hotspot/cpu/aarch64/vm_version_aarch64.cpp
@@ -161,9 +161,6 @@ void VM_Version::initialize() {
         (_model == CPU_MODEL_AMPERE_1A || _model == CPU_MODEL_AMPERE_1B)) {
       FLAG_SET_DEFAULT(CodeEntryAlignment, 32);
     }
-    if (FLAG_IS_DEFAULT(UseSignumIntrinsic)) {
-      FLAG_SET_DEFAULT(UseSignumIntrinsic, true);
-    }
   }
 
   // ThunderX
@@ -243,7 +240,7 @@ void VM_Version::initialize() {
     }
   }
 
-  if (_cpu == CPU_ARM) {
+  if (_features & (CPU_FP | CPU_ASIMD)) {
     if (FLAG_IS_DEFAULT(UseSignumIntrinsic)) {
       FLAG_SET_DEFAULT(UseSignumIntrinsic, true);
     }

Original file line number	Diff line number	Diff line change
`@@ -161,9 +161,6 @@ void VM_Version::initialize() {`
`161`	`161`	`(_model == CPU_MODEL_AMPERE_1A \|\| _model == CPU_MODEL_AMPERE_1B)) {`
`162`	`162`	`FLAG_SET_DEFAULT(CodeEntryAlignment, 32);`
`163`	`163`	`}`
`164`		`- if (FLAG_IS_DEFAULT(UseSignumIntrinsic)) {`
`165`		`- FLAG_SET_DEFAULT(UseSignumIntrinsic, true);`
`166`		`- }`
`167`	`164`	`}`
`168`	`165`
`169`	`166`	`// ThunderX`
`@@ -243,7 +240,7 @@ void VM_Version::initialize() {`
`243`	`240`	`}`
`244`	`241`	`}`
`245`	`242`
`246`		`- if (_cpu == CPU_ARM) {`
	`243`	`+ if (_features & (CPU_FP \| CPU_ASIMD)) {`
`247`	`244`	`if (FLAG_IS_DEFAULT(UseSignumIntrinsic)) {`
`248`	`245`	`FLAG_SET_DEFAULT(UseSignumIntrinsic, true);`
`249`	`246`	`}`