[Openmp-commits] [PATCH] D130509: [Libomptarget] Reintroduce host architecture checks for device RTL

[Shilei Tian via Phabricator via Openmp-commits]

tianshilei1992 added inline comments.


================
Comment at: openmp/libomptarget/DeviceRTL/CMakeLists.txt:22
+# Check to ensure the host system is a supported host architecture.
+if(NOT CMAKE_HOST_SYSTEM_PROCESSOR MATCHES "(x86_64|ppc64le|aarch64)")
+  libomptarget_say("Not building DeviceRTL: unknown host arch: ${CMAKE_HOST_SYSTEM_PROCESSOR}")
----------------
I remember there is a bug report about the name of host arch is not consistent among different OSs. If we just want to disable build for 32-bit system, we might want something else.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D130509/new/

https://reviews.llvm.org/D130509