<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><br><div><div>On Jan 23, 2013, at 2:26 AM, İsmail Dönmez <<a href="mailto:ismail@donmez.ws">ismail@donmez.ws</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div dir="ltr">Hi;<br><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Jan 23, 2013 at 12:49 AM, Douglas Gregor <span dir="ltr"><<a href="mailto:dgregor@apple.com" target="_blank">dgregor@apple.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin: 0px 0px 0px 0.8ex; border-left-width: 1px; border-left-color: rgb(204, 204, 204); border-left-style: solid; padding-left: 1ex; position: static; z-index: auto; ">Author: dgregor<br>
Date: Tue Jan 22 17:49:45 2013<br>
New Revision: 173208<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=173208&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=173208&view=rev</a><br>
Log:<br>
Fix compilation on Linux, which defines PATH_MAX in a weird place,<br>
from Saleem Abdulrasool!<br>
<br>
Modified:<br>
cfe/trunk/lib/Lex/HeaderSearch.cpp<br>
cfe/trunk/lib/Lex/ModuleMap.cpp<br>
<br>
Modified: cfe/trunk/lib/Lex/HeaderSearch.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Lex/HeaderSearch.cpp?rev=173208&r1=173207&r2=173208&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Lex/HeaderSearch.cpp?rev=173208&r1=173207&r2=173208&view=diff</a><br>
==============================================================================<br>
--- cfe/trunk/lib/Lex/HeaderSearch.cpp (original)<br>
+++ cfe/trunk/lib/Lex/HeaderSearch.cpp Tue Jan 22 17:49:45 2013<br>
@@ -23,6 +23,11 @@<br>
#include "llvm/Support/FileSystem.h"<br>
#include "llvm/Support/Path.h"<br>
#include <cstdio><br>
+#if defined(LLVM_ON_UNIX)<br>
+#if defined(__linux__)<br>
+#include <linux/limits.h><br>
+#endif<br>
+#endif<br>
using namespace clang;<br></blockquote><div><br></div><div style="">Headers under linux/* is not for userspace consumption. The correct header file is <limits.h>.</div><div style=""><br></div></div></div></div>
</blockquote><br></div><div>According to the thread for this patch ([PATCH] clang: fix compilation on Linux), <limits.h> doesn't pull in PATH_MAX. I don't have a Linux system to verify either way.</div><br><div><span class="Apple-tab-span" style="white-space:pre"> </span>- Doug</div></body></html>