[PATCH] Add FreeBSD support to sanitizers' procmaps facilities

Viktor Kutuzov vkutuzov at accesssoftek.com
Tue Mar 18 03:42:24 PDT 2014



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

================
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;
----------------
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.


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



More information about the llvm-commits mailing list