[PATCH] D40898: [Sanitizers] Basic sanitizer Solaris support (PR 33274)

[Rainer Orth via Phabricator via llvm-commits]

ro marked an inline comment as done.
ro added inline comments.


================
Comment at: lib/sanitizer_common/sanitizer_internal_defs.h:144
 #endif
+#if SANITIZER_SOLARIS && !defined(_LP64)
+typedef long pid_t;
----------------
krytarowski wrote:
> Are there still !LP64 Solaris systems?
Sure: while there are no 32-bit Solaris kernels beyond Solaris 10
(maybe in the OpenSolaris/Illumos space; I don't know), every
Solaris system can execute both 32 and 64-bit programs.
In fact, this first version of the port is 32-bit x86 only at the
moment, though this will change.


Repository:
  rCRT Compiler Runtime

https://reviews.llvm.org/D40898