[Lldb-commits] [PATCH] D85265: Add a setting to always run all threads when stepping

[Jim Ingham via Phabricator via lldb-commits]

jingham added inline comments.


================
Comment at: lldb/source/Commands/CommandObjectThread.cpp:486-490
+      // NonStopMode runs all threads down in the ProcessPlugin layer, but
+      // at this level we need to pretend we are actually only running this
+      // thread.  So functionally it does the same thing as
+      // GetSteppingRunsAllThreads.  So it overrides the runs all threads
+      // setting.
----------------
friss wrote:
> It feels like the last sentence shouldn't be there.
> 
> Also, the comment doesn't explain *why* "we need to pretend". 
Is this better?  


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D85265/new/

https://reviews.llvm.org/D85265