[Lldb-commits] [PATCH] D31374: Add support for tracing hello-world application on NetBSD

2017-03-28 Thread Phabricator via Phabricator via lldb-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL298953: Add support for tracing hello-world application on NetBSD (authored by kamil). Changed prior to commit: https://reviews.llvm.org/D31374?vs=93312=93315#toc Repository: rL LLVM

[Lldb-commits] [PATCH] D31374: Add support for tracing hello-world application on NetBSD

2017-03-28 Thread Kamil Rytarowski via Phabricator via lldb-commits
krytarowski updated this revision to Diff 93312. krytarowski added a comment. Apply changes from review. Repository: rL LLVM https://reviews.llvm.org/D31374 Files: source/Plugins/Process/NetBSD/CMakeLists.txt source/Plugins/Process/NetBSD/NativeProcessNetBSD.cpp

[Lldb-commits] [PATCH] D31374: Add support for tracing hello-world application on NetBSD

2017-03-28 Thread Kamil Rytarowski via Phabricator via lldb-commits
krytarowski added a comment. This code is used as a base for further improvements, I'm going to commit it... debugging hello world still works. $ lldb ./hello

[Lldb-commits] [PATCH] D31374: Add support for tracing hello-world application on NetBSD

2017-03-28 Thread Kamil Rytarowski via Phabricator via lldb-commits
krytarowski added a comment. In https://reviews.llvm.org/D31374#712221, @labath wrote: > I wasn't able to go into this too deeply, but here is what I have after a > quick pass. I won't be able to review this thoroughly that soon, but I think > it can go in after you take my comments into

[Lldb-commits] [PATCH] D31374: Add support for tracing hello-world application on NetBSD

2017-03-28 Thread Pavel Labath via Phabricator via lldb-commits
labath added a comment. I wasn't able to go into this too deeply, but here is what I have after a quick pass. I won't be able to review this thoroughly that soon, but I think it can go in after you take my comments into consideration. Comment at:

[Lldb-commits] [PATCH] D31374: Add support for tracing hello-world application on NetBSD

2017-03-26 Thread Kamil Rytarowski via Phabricator via lldb-commits
krytarowski added a comment. Right now I'm not sure how to optimize reading GPR. If possible I would reschedule it for later. Repository: rL LLVM https://reviews.llvm.org/D31374 ___ lldb-commits mailing list lldb-commits@lists.llvm.org

[Lldb-commits] [PATCH] D31374: Add support for tracing hello-world application on NetBSD

2017-03-26 Thread Kamil Rytarowski via Phabricator via lldb-commits
krytarowski added a comment. In https://reviews.llvm.org/D31374#710864, @kettenis wrote: > On OpenBSD the register context used in core dumps uses the same layout as > ptrace(2). That's not the case on all OSes, but I believe that is the case > for NetBSD as well. Would it be possible to

[Lldb-commits] [PATCH] D31374: Add support for tracing hello-world application on NetBSD

2017-03-26 Thread Mark Kettenis via Phabricator via lldb-commits
kettenis added a comment. On OpenBSD the register context used in core dumps uses the same layout as ptrace(2). That's not the case on all OSes, but I believe that is the case for NetBSD as well. Would it be possible to re-use the register context data structures defined in the

[Lldb-commits] [PATCH] D31374: Add support for tracing hello-world application on NetBSD

2017-03-26 Thread Kamil Rytarowski via Phabricator via lldb-commits
krytarowski created this revision. krytarowski added a project: LLDB. Herald added a subscriber: mgorny. This patch is a stripped down from features a NetBSD process code (patch is kept under 2k LOC). This code has assumption that there is only one thread within a debugged process. The only