[PATCH] D24332: [ESan] [MIPS] Fix workingset-signal-posix.cpp on MIPS

[Sagar Thakur via llvm-commits]

slthakur added inline comments.

================
Comment at: lib/sanitizer_common/sanitizer_platform_limits_posix.h:610
@@ -609,3 +609,3 @@
   struct __sanitizer_kernel_sigset_t {
-    u8 sig[16];
+    uptr sig[2];
   };
----------------
bruening wrote:
> Aren't there references in source code files that need to be updated for this change?  At a glance I see things like "(sizeof(k_set->sig[0]) * 8)".
There the two functions which reference __sanitizer_kernel_sigset_t.sig:

- internal_sigdelset
- internal_sigismember

Both of these functions were producing wrong results on big endian machines because __sanitizer_kernel_sigset_t.sig was declared as a char array. Both these functions return correct result with this change.


Repository:
  rL LLVM

https://reviews.llvm.org/D24332