[Lldb-commits] [PATCH] D80112: Check if thread was suspended during previous stop added.

Ilya Bukonkin via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Wed Jun 10 13:55:55 PDT 2020


fallkrum updated this revision to Diff 269963.
fallkrum added a comment.

Sorry for misunderstanding, you were right, it is possible to reproduce situation via SB API tests. Please check them out.

In D80112#2080516 <https://reviews.llvm.org/D80112#2080516>, @jingham wrote:

> This patch has gotten hard to read because the latest version seems to have lots of unrelated changes, maybe from running clang-format over code you aren't actually changing?  Can you remove all these irrelevant changes and repost the patch?


Yes, it was due to clang-format, removed irrelevant changes.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80112

Files:
  lldb/include/lldb/Target/Process.h
  lldb/source/Target/Process.cpp
  lldb/test/API/functionalities/thread/ignore_suspended/Makefile
  lldb/test/API/functionalities/thread/ignore_suspended/TestIgnoreSuspendedThread.py
  lldb/test/API/functionalities/thread/ignore_suspended/main.cpp
  lldb/unittests/CMakeLists.txt
  lldb/unittests/Process/CMakeLists.txt
  lldb/unittests/Process/ProcessEventDataTest.cpp
  lldb/unittests/Thread/CMakeLists.txt
  lldb/unittests/Thread/ThreadTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D80112.269963.patch
Type: text/x-patch
Size: 31143 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20200610/0d2ab058/attachment-0001.bin>


More information about the lldb-commits mailing list