[PATCH] D113717: [Symbolizer][Debuginfo] Add debuginfod client to llvm-symbolizer.

Nico Weber via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 10 07:26:09 PST 2021


thakis added a comment.

Turns out this isn't even just theoretical: This _did_ add a dependency on libcurl to lld:

  //lld/tools/lld:lld has an assert_no_deps entry:
    //llvm/lib/Debuginfod:Debuginfod
  which fails for the dependency path:
    //lld/tools/lld:lld ->
    //lld/COFF:COFF ->
    //llvm/lib/DebugInfo/Symbolize:Symbolize ->
    //llvm/lib/Debuginfod:Debuginfod

The linker shouldn't do http requests, obviously. Let's revert this for now until at least the lld dep is figured out.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D113717



More information about the llvm-commits mailing list