[PATCH] Add FreeBSD support to sanitizers' procmaps facilities

Alexey Samsonov samsonov at google.com
Tue Mar 18 04:06:45 PDT 2014



================
Comment at: lib/sanitizer_common/sanitizer_procmaps_linux.cc:285
@@ -229,3 +284,3 @@
 
 void GetMemoryProfile(fill_profile_f cb, uptr *stats, uptr stats_size) {
   char *smaps = 0;
----------------
Viktor Kutuzov wrote:
> Alexey Samsonov wrote:
> > I doubt this function will work on FreeBSD as expected.
> I propose that we add FreeBSD support to this function in another diff addressed to a specific failure so we can be sure it works as expected.
OK

================
Comment at: lib/sanitizer_common/sanitizer_procmaps_linux.cc:20
@@ +19,3 @@
+#if SANITIZER_FREEBSD
+#include <unistd.h>
+#include <sys/sysctl.h>
----------------
Viktor Kutuzov wrote:
> Alexey Samsonov wrote:
> > Please sort the headers.
> That will cause compilation errors; many system headers do not respect their dependencies on FreeBSD.
OK, disregard this comment then.


http://llvm-reviews.chandlerc.com/D3100



More information about the llvm-commits mailing list