protze.joachim added a comment.
I think in openmp/tools currently only libraries and header files get
installed, but no binaries.
I suggest to define `OPENMP_TOOLS_INSTALL_BINDIR`,
`OPENMP_TOOLS_INSTALL_LIBDIR`, and `OPENMP_TOOLS_INSTALL_INCLUDEDIR` and
probably base these variables on OPENMP_TOOLS_INSTALL_DIR, if defined (e.g.,
for distro builds), or OPENMP_INSTALL_{BIN|LIB|INCLUDE}DIR as fallback.
openmp/tools/*/CMakeLists.txt should then use these variables instead of the
various variables they use currently.
================
Comment at: openmp/tools/CMakeLists.txt:1-3
+set(OPENMP_TOOLS_INSTALL_DIR "${CMAKE_INSTALL_BINDIR}" CACHE PATH
+ "Path for binary subdirectory (defaults to '${CMAKE_INSTALL_BINDIR}')")
+mark_as_advanced(OPENMP_TOOLS_INSTALL_DIR)
----------------
Is this variable used anywhere?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D117977/new/
https://reviews.llvm.org/D117977
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
