[Lldb-commits] [PATCH] D62879: [CMake] Add configuration dirs as potential locations for llvm-lit and llvm-tblgen in standalone builds

[Stella Stamenova via Phabricator via lldb-commits]

stella.stamenova added inline comments.


================
Comment at: lldb/cmake/modules/LLDBStandalone.cmake:67
+    append_configuration_directories(${LLVM_TOOLS_BINARY_DIR} config_dirs)
+    find_program(LLVM_TABLEGEN_EXE "llvm-tblgen" ${config_dirs} 
NO_DEFAULT_PATH)
   endif()
----------------
sgraenitz wrote:
> How is this working on Windows anyway? It should be `llvm-tblgen.exe` there 
> right?
Ah, good point. You should use `llvm-tblgen${HOST_EXECUTABLE_SUFFIX}` as the 
name. That will add the suffix when required.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D62879



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits