[PATCH] D114415: [llvm] [Debuginfo] Add HTTP Server to Debuginfod library.

David Blaikie via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 17 20:32:48 PST 2022


dblaikie added a comment.

In D114415#3250033 <https://reviews.llvm.org/D114415#3250033>, @noajshu wrote:

> @dblaikie Thanks for taking another look and the LGTM! Per your suggestion I've added a test fixture to call the `HTTPClient` initialization and teardown methods. Thanks for all the comments on this patch.
>
> It seems at first `llvm-debuginfod` will be the sole user tool. So, my inclination is to hold off on committing this until the llvm-debuginfod server (D114845 <https://reviews.llvm.org/D114845> and D114846 <https://reviews.llvm.org/D114846>) is ready to land.

Yep, sounds fair.


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

https://reviews.llvm.org/D114415



More information about the llvm-commits mailing list