[Openmp-commits] [PATCH] D89426: [libomptarget] Require LLVM source tree to build libomptarget

[Jon Chesterfield via Phabricator via Openmp-commits]

JonChesterfield added a comment.

Answering my own questions about cmake semantics by trial and error, the above revision handles the missing definition by not building libomptarget.

Tested the various permutations of LLVM_MAIN_INCLUDE_DIR present/absent in each file locally and all behave as one would hope.

Passing -DOPENMP_ENABLE_LIBOMPTARGET wins out over this default. Picked 'status' as that is used with message elsewhere in openmp's cmake.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D89426