[all-commits] [llvm/llvm-project] 933d3e: [lldb] Drop RegisterInfoInterface::GetDynamicRegis...

Pavel Labath via All-commits all-commits at lists.llvm.org
Wed Apr 5 04:26:04 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 933d3ee60007f5798319cad05b981cb265578ba0
      https://github.com/llvm/llvm-project/commit/933d3ee60007f5798319cad05b981cb265578ba0
  Author: Pavel Labath <pavel at labath.sk>
  Date:   2023-04-05 (Wed, 05 Apr 2023)

  Changed paths:
    M lldb/source/Plugins/Process/Linux/NativeRegisterContextLinux_x86_64.cpp
    M lldb/source/Plugins/Process/Linux/NativeRegisterContextLinux_x86_64.h
    M lldb/source/Plugins/Process/Utility/RegisterContextLinux_i386.cpp
    M lldb/source/Plugins/Process/Utility/RegisterContextLinux_i386.h
    M lldb/source/Plugins/Process/Utility/RegisterContextLinux_s390x.cpp
    M lldb/source/Plugins/Process/Utility/RegisterContextLinux_s390x.h
    A lldb/source/Plugins/Process/Utility/RegisterContextLinux_x86.h
    M lldb/source/Plugins/Process/Utility/RegisterContextLinux_x86_64.cpp
    M lldb/source/Plugins/Process/Utility/RegisterContextLinux_x86_64.h
    M lldb/source/Plugins/Process/Utility/RegisterInfoInterface.h

  Log Message:
  -----------
  [lldb] Drop RegisterInfoInterface::GetDynamicRegisterInfo

"Dynamic register info" is a very overloaded term, and this particular
instance of it was only used for passing the information about the
"orig_[re]ax" pseudo-register on x86 through some generic code. Since
both sides of the code are x86-specific, I have replaced this with a
more direct route.

Differential Revision: https://reviews.llvm.org/D147045


  Commit: af9e1fa178433653eb3d36c42cad016449873cfc
      https://github.com/llvm/llvm-project/commit/af9e1fa178433653eb3d36c42cad016449873cfc
  Author: Pavel Labath <pavel at labath.sk>
  Date:   2023-04-05 (Wed, 05 Apr 2023)

  Changed paths:
    M lldb/source/Target/ThreadPlan.cpp
    A lldb/test/API/functionalities/fork/resumes-child/Makefile
    A lldb/test/API/functionalities/fork/resumes-child/TestForkResumesChild.py
    A lldb/test/API/functionalities/fork/resumes-child/main.cpp

  Log Message:
  -----------
  [lldb] Detach the child process when stepping over a fork

Step over thread plans were claiming to explain the fork stop reasons,
which prevented the default fork logic (detaching from the child
process) from kicking in. This patch changes that.

Differential Revision: https://reviews.llvm.org/D141605


Compare: https://github.com/llvm/llvm-project/compare/8f5db5332b2a...af9e1fa17843


More information about the All-commits mailing list