[all-commits] [llvm/llvm-project] ab6394: [lldb] Bring Debuginfod's StreamedHTTPResponseHand...

[Stefan Gränitz via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: ab6394d21a595cf95cb1429a2f0e5779959abce1
      https://github.com/llvm/llvm-project/commit/ab6394d21a595cf95cb1429a2f0e5779959abce1
  Author: Stefan Gränitz <stefan.graenitz at gmail.com>
  Date:   2026-04-08 (Wed, 08 Apr 2026)

  Changed paths:
    M lldb/source/Plugins/SymbolLocator/SymStore/SymbolLocatorSymStore.cpp
    M lldb/test/API/symstore/TestSymStore.py
    A llvm/include/llvm/Support/HTTP/StreamedHTTPResponseHandler.h
    M llvm/lib/Debuginfod/Debuginfod.cpp
    M llvm/lib/Support/HTTP/CMakeLists.txt
    A llvm/lib/Support/HTTP/StreamedHTTPResponseHandler.cpp

  Log Message:
  -----------
  [lldb] Bring Debuginfod's StreamedHTTPResponseHandler to SymbolLocatorSymStore (#187687)

SymbolLocatorSymStore used a simple local implementation of
HTTPResponseHandler so far. That was fine for basic usage, but it would
cause issues down the line. This patch hoists the
StreamedHTTPResponseHandler class from libDebuginfod to SupportHTTP and
integrates it in SymbolLocatorSymStore. PDB file downloads will now be
buffered on disk, which is necessary since they can be huge.

We use the opportunity an stop logging 404 responses (file not found on
server) and print warnings for all other erroneous HTTP responses. It
was more complicated before, because the old response handler created
the underlying file in any case. The new one does that only once the
first content package comes in.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications