<div>Thanks, Mike.</div>
<div> </div>
<div>Can you give me some pointers on how to do a platform-specific file?  I don't see any examples, and I don't know cmake.  For example, have a "windows" subdirectory, and some conditional in the CMakeLists.txt file?  And how to make sure make doesn't pick up the file.</div>

<div> </div>
<div>I figure the Basic library might be a good place for it.</div>
<div> </div>
<div>Note that I found some errors in the registry part of it, and I'm experimenting with getting the windows.h include to work.</div>
<div> </div>
<div>> Aside from that, I think the rest of the patch has now been checked in.  If there are any pieces I missed, please svn up, and send in what's missing.  And welcome.</div>
<div> </div>
<div>You got them all, thanks!<br></div>
<div>-John<br></div>
<div class="gmail_quote">On Thu, Oct 8, 2009 at 4:33 PM, Mike Stump <span dir="ltr"><<a href="mailto:mrs@apple.com">mrs@apple.com</a>></span> wrote:<br>
<blockquote style="BORDER-LEFT: #ccc 1px solid; MARGIN: 0px 0px 0px 0.8ex; PADDING-LEFT: 1ex" class="gmail_quote">
<div class="im">On Oct 8, 2009, at 12:08 PM, John Thompson wrote:<br></div>
<div class="im">
<blockquote style="BORDER-LEFT: #ccc 1px solid; MARGIN: 0px 0px 0px 0.8ex; PADDING-LEFT: 1ex" class="gmail_quote"><br>include/clang/Frontend/InitHeaderSearch.h:<br>lib/Frontend/InitHeaderSearch.cpp:<br>tools/clang-cc/clang-cc.cpp:<br>
<br>This is a slight revision to the patch I posted yesterday to have the default include path code use the target triple, as the line numbers were off due to some other changes.  It also sets up include paths for VC++ and Cygwin headers, along with the existing MinGW headers, plus the newer 4.4.0 version.  Note that running the tests with the MinGW headers results in 40 or more failed tests.  Currently, with this patch, only 23 tests are failing with the VC++ headers.  I did leave in some disabled code regarding the question I posted yesterday about including windows.h for the registry stuff.<br>
</blockquote><br></div>I've checked this in, along with some minor whitespacing issues.  Please keep lines to 80 columns.  Also, be sure to detab the files as well.  In some editors, there is a setting to avoid the use of hard tabs.<br>
<br>I think the windows code should be boosted out to a new file, and hidden away from us mere mortals.  In general #if in files is to be discouraged.<br></blockquote></div><br><br clear="all">
<div></div><br>-- <br>John Thompson<br><a href="mailto:John.Thompson.JTSoftware@gmail.com">John.Thompson.JTSoftware@gmail.com</a><br><br>