<div class="gmail_extra"><div class="gmail_quote">On Fri, Apr 20, 2012 at 2:24 PM, Michael Spencer <span dir="ltr"><<a href="mailto:bigcheesegs@gmail.com" target="_blank">bigcheesegs@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
When building LLVM on Linux with libc++ with CMake TIME_WITH_SYS_TIME is<br>
undefined, and HAVE_SYS_TIME_H is defined. This ends up including<br>
sys/time.h but not time.h. Unix/TimeValue.inc requires time.h for asctime_r<br>
and localtime. libstdc++ seems to include time.h anyway, but libc++ does<br>
not.<br>
<br>
Fix this by always including time.h<br></blockquote><div><br></div><div>Seems fine to me...</div><div><br></div><div>Should we include time.h or ctime? I always forget which is preferred in LLVM.</div><div><br></div></div>
</div>