Author: sedymrak Date: 2024-08-05T11:23:57-07:00 New Revision: 248c53429427034f45705af60d47f3b1090c4799
URL: https://github.com/llvm/llvm-project/commit/248c53429427034f45705af60d47f3b1090c4799 DIFF: https://github.com/llvm/llvm-project/commit/248c53429427034f45705af60d47f3b1090c4799.diff LOG: [lldb] Delete StepScope enum whose values are ignored. (#101981) StepScope enum is a type whose values are passed around, but they are ultimately ignored. --------- Co-authored-by: Matej Košík <matej.ko...@codasip.com> Added: Modified: lldb/source/Commands/CommandObjectThread.cpp Removed: ################################################################################ diff --git a/lldb/source/Commands/CommandObjectThread.cpp b/lldb/source/Commands/CommandObjectThread.cpp index 366b6dd965b38..605f872a9f45e 100644 --- a/lldb/source/Commands/CommandObjectThread.cpp +++ b/lldb/source/Commands/CommandObjectThread.cpp @@ -247,8 +247,6 @@ class CommandObjectThreadBacktrace : public CommandObjectIterateOverThreads { CommandOptions m_options; }; -enum StepScope { eStepScopeSource, eStepScopeInstruction }; - #define LLDB_OPTIONS_thread_step_scope #include "CommandOptions.inc" @@ -374,15 +372,13 @@ class CommandObjectThreadStepWithTypeAndScope : public CommandObjectParsed { CommandObjectThreadStepWithTypeAndScope(CommandInterpreter &interpreter, const char *name, const char *help, const char *syntax, - StepType step_type, - StepScope step_scope) + StepType step_type) : CommandObjectParsed(interpreter, name, help, syntax, eCommandRequiresProcess | eCommandRequiresThread | eCommandTryTargetAPILock | eCommandProcessMustBeLaunched | eCommandProcessMustBePaused), - m_step_type(step_type), m_step_scope(step_scope), - m_class_options("scripted step") { + m_step_type(step_type), m_class_options("scripted step") { AddSimpleArgumentList(eArgTypeThreadIndex, eArgRepeatOptional); if (step_type == eStepTypeScripted) { @@ -621,7 +617,6 @@ class CommandObjectThreadStepWithTypeAndScope : public CommandObjectParsed { } StepType m_step_type; - StepScope m_step_scope; ThreadStepScopeOptionGroup m_options; OptionGroupPythonClassWithDict m_class_options; OptionGroupOptions m_all_options; @@ -2561,35 +2556,35 @@ CommandObjectMultiwordThread::CommandObjectMultiwordThread( interpreter, "thread step-in", "Source level single step, stepping into calls. Defaults " "to current thread unless specified.", - nullptr, eStepTypeInto, eStepScopeSource))); + nullptr, eStepTypeInto))); LoadSubCommand("step-out", CommandObjectSP(new CommandObjectThreadStepWithTypeAndScope( interpreter, "thread step-out", "Finish executing the current stack frame and stop after " "returning. Defaults to current thread unless specified.", - nullptr, eStepTypeOut, eStepScopeSource))); + nullptr, eStepTypeOut))); LoadSubCommand("step-over", CommandObjectSP(new CommandObjectThreadStepWithTypeAndScope( interpreter, "thread step-over", "Source level single step, stepping over calls. Defaults " "to current thread unless specified.", - nullptr, eStepTypeOver, eStepScopeSource))); + nullptr, eStepTypeOver))); LoadSubCommand("step-inst", CommandObjectSP(new CommandObjectThreadStepWithTypeAndScope( interpreter, "thread step-inst", "Instruction level single step, stepping into calls. " "Defaults to current thread unless specified.", - nullptr, eStepTypeTrace, eStepScopeInstruction))); + nullptr, eStepTypeTrace))); LoadSubCommand("step-inst-over", CommandObjectSP(new CommandObjectThreadStepWithTypeAndScope( interpreter, "thread step-inst-over", "Instruction level single step, stepping over calls. " "Defaults to current thread unless specified.", - nullptr, eStepTypeTraceOver, eStepScopeInstruction))); + nullptr, eStepTypeTraceOver))); LoadSubCommand( "step-scripted", @@ -2600,7 +2595,7 @@ CommandObjectMultiwordThread::CommandObjectMultiwordThread( "that will be used to populate an SBStructuredData Dictionary, which " "will be passed to the constructor of the class implementing the " "scripted step. See the Python Reference for more details.", - nullptr, eStepTypeScripted, eStepScopeSource))); + nullptr, eStepTypeScripted))); LoadSubCommand("plan", CommandObjectSP(new CommandObjectMultiwordThreadPlan( interpreter))); _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
