mgorny marked 2 inline comments as done.
mgorny added inline comments.
================
Comment at: lldb/test/Shell/Watchpoint/Inputs/thread-dbreg.c:1-23
+#include <pthread.h>
+
+int g_watchme = 0;
+
+void *thread_func(void *arg) {
+ /* watchpoint trigger from subthread */
+ g_watchme = 2;
----------------
labath wrote:
> Maybe simplify this and remove the threads and stuff?
Threads are intentional since new thread handler copies dbregs per the other
patch. This makes sure that new thread handler will not crash when it is unable
to set dbregs.
================
Comment at: lldb/test/Shell/Watchpoint/netbsd-nouserdbregs.test:5
+# REQUIRES: native && system-netbsd && (target-x86 || target-x86_64) &&
!dbregs-set
+# RUN: %clang %p/Inputs/thread-dbreg.c -pthread -g -o %t.out
+# RUN: %lldb -b -o 'settings set interpreter.stop-command-source-on-error
false' -s %s %t.out 2>&1 | FileCheck %s
----------------
labath wrote:
> You should use %clang_host now. Are you sure that this even works on current
> master?
I have to admit that I'm at ~Oct 31 in my checkout. I've tried using
`%clang_host` but it failed with `fg` saying that there is no job control.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D70050/new/
https://reviews.llvm.org/D70050
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
