[Openmp-commits] [PATCH] D101509: An attempt to abandon omptarget out-of-tree builds.

[Shilei Tian via Phabricator via Openmp-commits]

tianshilei1992 added inline comments.


================
Comment at: openmp/libomptarget/deviceRTLs/amdgcn/CMakeLists.txt:14
+# An installed LLVM package should have been discovered before.
+if (NOT LLVM_DIR)
   libomptarget_say("Not building AMDGCN device RTL: AOMP not found")
----------------
This check could be avoided if the include of `libomptarget` is guarded by the check of LLVM. That said, all CMake code in `libomptarget` can assume that LLVM is already there.


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

https://reviews.llvm.org/D101509