[PATCH] [sanitizer] Implement ioctl decoding.

Sergey Matveev earthdok at google.com
Wed Feb 5 11:17:06 PST 2014



================
Comment at: lib/sanitizer_common/sanitizer_platform_limits_posix.cc:790
@@ -789,1 +789,3 @@
 
+// We define those on Linux and Mac, but only check on Linux.
+COMPILER_CHECK(IOC_NRBITS == _IOC_NRBITS);
----------------
Evgeniy Stepanov wrote:
> Why? Missing headers?
I'm not sure if they're there or not. The pre-existing check was under an #ifdef LINUX. I just documented this state of affairs explicitly.

Also the EVIOC stuff seems to be Linux-specific.


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



More information about the llvm-commits mailing list