[PATCH] D113717: [Symbolizer][Debuginfo] Add debuginfod client to llvm-symbolizer.
David Blaikie via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Nov 29 12:36:39 PST 2021
dblaikie added a comment.
In D113717#3157634 <https://reviews.llvm.org/D113717#3157634>, @jhenderson wrote:
> Not looked at anything else in the patch stack, but I can more-or-less guess what is going on (storing the debug data on a remote server for the symbolizer and other tools to be able to look up?). If so, in general, this looks fine to me. My only slight concern is I wonder how many places (especially build bots) will exercise this test, since it requires a new dependency. Might be worth approaching some build bot owners and asking if they'd be willing to install curl as part of their setup.
I seem to recall some mention that this API was usable with a local cache even without the curl dependency? Perhaps a test using only a local cache (not needing a server or curl) could be added as well? (the test with curl/etc might be good to keep too - though perhaps it could be removed if the API functionality is already tested elsewhere - if it isn't/can't readily be tested elsewhere then here's fine too)
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D113717/new/
https://reviews.llvm.org/D113717
More information about the llvm-commits
mailing list