[Openmp-commits] [PATCH] D117977: [cmake] Don't export `LLVM_TOOLS_INSTALL_DIR` anymore

John Ericson via Phabricator via Openmp-commits openmp-commits at lists.llvm.org
Tue Jun 7 22:56:46 PDT 2022


Ericson2314 added a comment.
Herald added a subscriber: bzcheeseman.
Herald added a project: All.

I moved the `openmp` definitions so they should be used. Don't need the other variables yet I don't think?



================
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)
----------------
protze.joachim wrote:
> Is this variable used anywhere?
`llvm_add_tool` computes a usage.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D117977



More information about the Openmp-commits mailing list