[Openmp-commits] [PATCH] D145513: [Libomptarget] Update handling of architectures for DeviceRTL

[Ye Luo via Phabricator via Openmp-commits]

ye-luo added inline comments.


================
Comment at: openmp/libomptarget/DeviceRTL/CMakeLists.txt:68
+
+set(LIBOMPTARGET_DEVICE_ARCHITECTURES ${all_gpu_architectures} CACHE STRING
+    "List of device architectures to be used to compile the OpenMP DeviceRTL.")
----------------
Prefer default all instead of the full list.
When doing an incremental build and having new arch being listed. `all` continue to build everything but cached list doesn't 


================
Comment at: openmp/libomptarget/DeviceRTL/CMakeLists.txt:74
+elseif(LIBOMPTARGET_DEVICE_ARCHITECTURES STREQUAL "auto")
+  if (NOT LIBOMPTARGET_NVPTX_ARCH AND NOT LIBOMPTARGET_AMDGPU_ARCH)
+    libomptarget_error_say(
----------------
Are both tool always built together?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D145513