[all-commits] [llvm/llvm-project] 799e90: [lldb] Create a default rate limit constant in Pro...

Jonas Devlieghere via All-commits all-commits at lists.llvm.org
Mon Mar 31 08:29:43 PDT 2025


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 799e9053641a6478d3144866a97737b37b87c260
      https://github.com/llvm/llvm-project/commit/799e9053641a6478d3144866a97737b37b87c260
  Author: Jonas Devlieghere <jonas at devlieghere.com>
  Date:   2025-03-31 (Mon, 31 Mar 2025)

  Changed paths:
    M lldb/include/lldb/Core/Progress.h
    M lldb/source/Plugins/SymbolFile/DWARF/ManualDWARFIndex.cpp
    M lldb/source/Plugins/SymbolFile/DWARF/SymbolFileDWARFDebugMap.cpp

  Log Message:
  -----------
  [lldb] Create a default rate limit constant in Progress (NFC) (#133506)

In #133211, Greg suggested making the rate limit configurable through a
setting. Although adding the setting is easy, the two places where we
currently use rate limiting aren't tied to a particular debugger.
Although it'd be possible to hook up, given how few progress events
currently implement rate limiting, I don't think it's worth threading
this through, if that's even possible.

I still think it's a good idea to be consistent and make it easy to pick
the same rate limiting value, so I've moved it into a constant in the
Progress class.



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


More information about the All-commits mailing list