[all-commits] [llvm/llvm-project] a261ee: [lldb] Store *signed* ranges in lldb_private::Bloc...

[Pavel Labath via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: a261eee61200cb6aa3eac0e7dc03940a6afd7d54
      https://github.com/llvm/llvm-project/commit/a261eee61200cb6aa3eac0e7dc03940a6afd7d54
  Author: Pavel Labath <pavel at labath.sk>
  Date:   2025-01-09 (Thu, 09 Jan 2025)

  Changed paths:
    M lldb/include/lldb/Symbol/Block.h
    M lldb/unittests/Utility/RangeMapTest.cpp

  Log Message:
  -----------
  [lldb] Store *signed* ranges in lldb_private::Block (#120224)

This is to support functions whose entry points aren't their lowest
address

(https://discourse.llvm.org/t/rfcish-support-for-discontinuous-functions/83244).
The alternative is to keep blocks relative to the lowest address, but
then introduce a separate concept for the function entry point, which I
think would be more confusing.

This patch just changes the type signedness, it doesn't create any
negative offsets yet. Since combining values with different signs can
sometimes produce unexpected results, and since this is the first use of
RangeVector with a signed type, I'm adding a test to verify that at
least the core functionality works correctly.



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