[Lldb-commits] [PATCH] D12659: Use SI_KERNEL on platforms defining it

Kamil Rytarowski via lldb-commits lldb-commits at lists.llvm.org
Thu Sep 10 16:46:30 PDT 2015


krytarowski updated this revision to Diff 34510.
krytarowski added a comment.

Rework the comment regarding SI_KERNEL.


Repository:
  rL LLVM

http://reviews.llvm.org/D12659

Files:
  source/Plugins/Process/POSIX/CrashReason.cpp

Index: source/Plugins/Process/POSIX/CrashReason.cpp
===================================================================
--- source/Plugins/Process/POSIX/CrashReason.cpp
+++ source/Plugins/Process/POSIX/CrashReason.cpp
@@ -28,11 +28,12 @@
 
     switch (info.si_code)
     {
+#ifdef SI_KERNEL
     case SI_KERNEL:
-        // Linux will occasionally send spurious SI_KERNEL codes.
-        // (this is poorly documented in sigaction)
+        // Some platforms will occasionally send nonstandard spurious SI_KERNEL codes.
         // One way to get this is via unaligned SIMD loads.
         return CrashReason::eInvalidAddress; // for lack of anything better
+#endif
     case SEGV_MAPERR:
         return CrashReason::eInvalidAddress;
     case SEGV_ACCERR:


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D12659.34510.patch
Type: text/x-patch
Size: 759 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20150910/08f9bddc/attachment.bin>


More information about the lldb-commits mailing list