[PATCH] D136702: [llvm-cov] Look up object files using debuginfod

Petr Hosek via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 25 23:46:08 PST 2023


phosek added inline comments.


================
Comment at: compiler-rt/test/profile/lit.site.cfg.py.in:7
 config.target_arch = "@PROFILE_TEST_TARGET_ARCH@"
+config.have_curl = @LLVM_ENABLE_CURL@
 
----------------
You need to pythonize this variable. For this to work with the runtimes build, we also need to export `LLVM_ENABLE_CURL` in `LLVMConfig.cmake`, see https://github.com/llvm/llvm-project/blob/7f48154ca11fee6579fe29a51ab2a329424d31c4/llvm/cmake/modules/LLVMConfig.cmake.in#L81

There's a broader question of whether we should be passing the variables like `LLVM_ENABLE_CURL` to the runtimes in the first build since they are only relevant to the LLVM build. A better approach would be to provide some indication whether debuginfod is available in the `llvm-cov --help` output (or perhaps have a dedicated flag) and then parse that information when running compiler-rt CMake build and use the result here.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D136702



More information about the llvm-commits mailing list