HomePhabricator

[lldb/API] Move SBCommandInterpreterRunOption in its own header. (NFC)

Authored by JDevlieghere on Apr 30 2020, 1:28 PM.

Description

[lldb/API] Move SBCommandInterpreterRunOption in its own header. (NFC)

Currently, SBCommandInterpreterRunOptions is defined in
SBCommandInterpreter.h. Given that the options are always passed by
reference, a forward declaration is sufficient.

That's not the case for SBCommandInterpreterRunResults, which we need
for a new overload for RunCommandInterpreter and that returns this new
class by value. We can't include SBCommandInterpreter.h because
SBCommandInterpreter::GetDebugger() returns SBDebugger by value and
therefore needs a full definition.

This patch moves the definition of SBCommandInterpreterRunOptions into
a new header. In a later patch, SBCommandInterpreterRunResults will
be defined in there as well, solving the aforementioned problem.

Differential revision: https://reviews.llvm.org/D79115