<p dir="ltr">OK. It would be nice to enable these checks on Mac later. Please change the comment into a FIXME and commit.</p>
<div class="gmail_quote">On Feb 5, 2014 11:16 PM, "Sergey Matveev" <<a href="mailto:earthdok@google.com">earthdok@google.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
<br>
================<br>
Comment at: lib/sanitizer_common/sanitizer_platform_limits_posix.cc:790<br>
@@ -789,1 +789,3 @@<br>
<br>
+// We define those on Linux and Mac, but only check on Linux.<br>
+COMPILER_CHECK(IOC_NRBITS == _IOC_NRBITS);<br>
----------------<br>
Evgeniy Stepanov wrote:<br>
> Why? Missing headers?<br>
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.<br>
<br>
Also the EVIOC stuff seems to be Linux-specific.<br>
<br>
<br>
<a href="http://llvm-reviews.chandlerc.com/D2689" target="_blank">http://llvm-reviews.chandlerc.com/D2689</a><br>
</blockquote></div>