[Lldb-commits] [lldb] [LLDB] Support all the Generic (Negative) SI Codes. (PR #140150)

[Jacob Lalonde via lldb-commits]

================
@@ -584,22 +584,26 @@ Status ELFLinuxSigInfo::Parse(const DataExtractor &data, const ArchSpec &arch,
   // 64b ELF have a 4 byte pad.
   if (data.GetAddressByteSize() == 8)
     offset += 4;
-  // Not every stop signal has a valid address, but that will get resolved in
-  // the unix_signals.GetSignalDescription() call below.
-  if (unix_signals.GetShouldStop(si_signo)) {
+
+  if (si_code < 0) {
+    sifields.kill.pid = data.GetU32(&offset);
+    sifields.kill.uid = data.GetU32(&offset);
----------------
Jlalond wrote:

No, the data extractor will increment the offset for each call, so the first call will take the first 0-4 bytes, and the second call the next 4-8 bytes.

https://github.com/llvm/llvm-project/pull/140150