[all-commits] [llvm/llvm-project] 6b3b6a: [lldb][NFC] Create StackID::IsYounger (#209402)

[Felipe de Azevedo Piovezan via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 6b3b6a88905cfb7e098d3c4a790ae8a196cfeab2
      https://github.com/llvm/llvm-project/commit/6b3b6a88905cfb7e098d3c4a790ae8a196cfeab2
  Author: Felipe de Azevedo Piovezan <fpiovezan at apple.com>
  Date:   2026-07-15 (Wed, 15 Jul 2026)

  Changed paths:
    M lldb/include/lldb/Target/StackID.h
    M lldb/source/Target/StackFrameList.cpp
    M lldb/source/Target/StackID.cpp
    M lldb/source/Target/ThreadPlanStepInstruction.cpp
    M lldb/source/Target/ThreadPlanStepOut.cpp
    M lldb/source/Target/ThreadPlanStepRange.cpp
    M lldb/source/Target/ThreadPlanStepUntil.cpp

  Log Message:
  -----------
  [lldb][NFC] Create StackID::IsYounger (#209402)

Currently, StackIDs are compared with a custom operator <. There are
some issues with that:
1. It's not clear what "<" means in this context. It really is a test of
"frame A is on top of frame B or not comparable". But this notion is not
expressed through "<".
2. This is not a real operator "<" in the sense that if "!<" does not
imply ">=". In particular, frames may not always be comparable (i.e.
(they are not part of the same stack).

To address these issues, this commit replaces `<` with a custom function
"IsYounger", which makes explicit what is being tested.



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