[llvm] [BOLT][RUNTIME][NFC] Fix aarch64 match (PR #100866)

Vladislav Khmelevsky via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 5 02:51:45 PDT 2024


================
@@ -82,7 +82,7 @@ endforeach()
 
 set(BOLT_ENABLE_RUNTIME_default OFF)
 if ((CMAKE_SYSTEM_PROCESSOR STREQUAL "x86_64"
-    OR CMAKE_SYSTEM_PROCESSOR STREQUAL "aarch64")
+    OR CMAKE_SYSTEM_PROCESSOR MATCHES "^(armv[8-9].*|arm64|aarch64)$")
----------------
yota9 wrote:

Thanks for your investigation @peterwaller-arm ! I'm not too deep in this field and I agree that most likely arm64/aarch64 would be enough, I just thought that armv8 - armv9 check won't hurt anyone - that is why I include them. Also I remember such checks in other  projects I saw. As for arm check - I'm not convinced it is need, not only I didn't see it anywhere, but it also might give a wrong idea/false positive check for 32 bit ARM CPUs, so I'm currently not convinced we need to include such check here..

https://github.com/llvm/llvm-project/pull/100866


More information about the llvm-commits mailing list