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

[Joseph Huber via Phabricator via Openmp-commits]

jhuber6 updated this revision to Diff 447456.
jhuber6 added a comment.

Updating to use sizeof pointer instead to elimate support for 32-bit hosts.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D130509

Files:
  openmp/libomptarget/DeviceRTL/CMakeLists.txt


Index: openmp/libomptarget/DeviceRTL/CMakeLists.txt
===================================================================
--- openmp/libomptarget/DeviceRTL/CMakeLists.txt
+++ openmp/libomptarget/DeviceRTL/CMakeLists.txt
@@ -18,6 +18,12 @@
   return()
 endif()
 
+# Check to ensure the host system is a supported host architecture.
+if(NOT ${CMAKE_SIZEOF_VOID_P} EQUAL "8")
+  libomptarget_say("Not building DeviceRTL: Runtime does not support 32-bit hosts")
+  return()
+endif()
+
 if (LLVM_DIR)
   # Builds that use pre-installed LLVM have LLVM_DIR set.
   # A standalone or LLVM_ENABLE_RUNTIMES=openmp build takes this route


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D130509.447456.patch
Type: text/x-patch
Size: 625 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/openmp-commits/attachments/20220725/326c8074/attachment.bin>