This revision was automatically updated to reflect the committed changes.
Closed by commit rL283548: Fix build failure on lldb-amd64-ninja-freebsd11
error caused by rL283474 (authored by valentinagiusti).
Changed prior to commit:
https://reviews.llvm.org/D25362?vs=73909=73945#toc
Repository:
emaste accepted this revision.
emaste added a reviewer: emaste.
emaste added a comment.
This revision is now accepted and ready to land.
Not tested, but looks reasonable to me.
I think we should avoid `siginfo_t` altogether here in order to support
cross-debugging, but that's for a different
valentinagiusti updated this revision to Diff 73909.
valentinagiusti added a comment.
cleaned up code
https://reviews.llvm.org/D25362
Files:
source/Plugins/Process/POSIX/CrashReason.cpp
source/Plugins/Process/POSIX/CrashReason.h
Index: source/Plugins/Process/POSIX/CrashReason.h
valentinagiusti updated this revision to Diff 73907.
valentinagiusti added a comment.
applied clang-format
https://reviews.llvm.org/D25362
Files:
source/Plugins/Process/POSIX/CrashReason.cpp
source/Plugins/Process/POSIX/CrashReason.h
Index: source/Plugins/Process/POSIX/CrashReason.h
valentinagiusti created this revision.
valentinagiusti added reviewers: zturner, labath.
valentinagiusti added subscribers: lldb-commits, emaste.
https://reviews.llvm.org/D25362
Files:
source/Plugins/Process/POSIX/CrashReason.cpp
source/Plugins/Process/POSIX/CrashReason.h
Index: