[Lldb-commits] [lldb] [llvm] [lldb] Add step back single instruction for targets supporting reverse execution (PR #191183)

[Maarten Steevens via lldb-commits]

================
@@ -2774,6 +2782,13 @@ CommandObjectMultiwordThread::CommandObjectMultiwordThread(
                      "Defaults to current thread unless specified.",
                      nullptr, eStepTypeTrace)));
 
+  LoadSubCommand("step-back-inst",
----------------
MaartenS11 wrote:

> The other would be to add another OptionGroup with -R and -F, and only add that option group if allows_reverse is passed in as true. Doing it this way would allow you to share this option group with thread continue.

I decided to do it this way, so now there is a new `OptionGroupDirection` used by both `thread step-inst` and `process continue`. It was a bit tricky sometimes trying to get the option sets right since `-F` and `-R` are each in their own set, but each set still needs all the other stepping options. I also hope the options don't conflict with the reverse stepping logic in a way I haven't thought of yet.

https://github.com/llvm/llvm-project/pull/191183