[PATCH] Update config.* files

Renato Golin renato.golin at linaro.org
Wed Feb 12 09:54:28 PST 2014


On 12 February 2014 16:26, Renato Golin <renato.golin at linaro.org> wrote:
> My lit ExecutionEngine tests are failing because they don'e recognize
> "aarch64", so, maybe changing in the CMake to capitalise?

Lit tests are passing with the change below, but they still need the
change in config.guess/sub for CMake to even detect the arch.

I'm not sure why this is necessary for AArch64 and it's not for ARM,
since both are capitalised.

configure does that automatically...

diff --git a/cmake/modules/AddLLVM.cmake b/cmake/modules/AddLLVM.cmake
index 59add8d..8ce1b8d 100644
--- a/cmake/modules/AddLLVM.cmake
+++ b/cmake/modules/AddLLVM.cmake
@@ -529,6 +529,9 @@ function(configure_lit_site_cfg input output)

    set(HOST_OS ${CMAKE_SYSTEM_NAME})
   set(HOST_ARCH ${CMAKE_SYSTEM_PROCESSOR})
+  if (HOST_ARCH STREQUAL "aarch64")
+    set(HOST_ARCH "AArch64")
+  endif()

   if (CLANG_ENABLE_ARCMT)
     set(ENABLE_CLANG_ARCMT "1")

ideas?

--renato



More information about the llvm-commits mailing list