[all-commits] [llvm/llvm-project] a5ab1d: [lldb] Add step target to ThreadPlanStepInRange co...

Dave Lee via All-commits all-commits at lists.llvm.org
Thu Feb 11 14:57:55 PST 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: a5ab1dc4ad2c02510e363b4dd3c267f9eaf11516
      https://github.com/llvm/llvm-project/commit/a5ab1dc4ad2c02510e363b4dd3c267f9eaf11516
  Author: Dave Lee <davelee.com at gmail.com>
  Date:   2021-02-11 (Thu, 11 Feb 2021)

  Changed paths:
    M lldb/include/lldb/Target/ThreadPlanStepInRange.h
    M lldb/source/Plugins/LanguageRuntime/CPlusPlus/CPPLanguageRuntime.cpp
    M lldb/source/Target/Thread.cpp
    M lldb/source/Target/ThreadPlanStepInRange.cpp

  Log Message:
  -----------
  [lldb] Add step target to ThreadPlanStepInRange constructor

`QueueThreadPlanForStepInRange` accepts a `step_into_target`, but the constructor for
`ThreadPlanStepInRange` does not. Instead, a caller would optionally call
`SetStepInTarget()` in a separate statement.

This change adds `step_into_target` as a constructor argument. This simplifies
construction of `ThreadPlanSP`, by avoiding a subsequent downcast and conditional
assignment. This constructor is already used in downstream repos.

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




More information about the All-commits mailing list