<div>> Second, should this really go in InitHeaderSearch?  Would it make more sense to put in the getDirectory call itself?<br></div>
<div>Yes.  The enclosed is probably better.<br></div>
<div>-John<br></div>
<div class="gmail_quote">On Thu, Dec 17, 2009 at 2:41 PM, Chris Lattner <span dir="ltr"><<a href="mailto:clattner@apple.com">clattner@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"><br>On Dec 17, 2009, at 1:08 PM, John Thompson wrote:<br><br></div>
<div class="im">> The stat function on Windows doesn't like a path with a trailing separator like:  ../../<br>> This patch works around thant, but I don't know if it's the optimal solution.<br><br></div>

<div>
<div></div>
<div class="h5">Two issues.  First, that idiom for using endswith currently generates gross code.  We should fix this (<a href="http://llvm.org/bugs/show_bug.cgi?id=5822" target="_blank">http://llvm.org/bugs/show_bug.cgi?id=5822</a>), but until then, please just check MappedPath.back() == '/' or something.<br>
<br>Second, should this really go in InitHeaderSearch?  Would it make more sense to put in the getDirectory call itself?<br><br>-Chris</div></div></blockquote></div><br><br clear="all"><br>-- <br>John Thompson<br><a href="mailto:John.Thompson.JTSoftware@gmail.com">John.Thompson.JTSoftware@gmail.com</a><br>
<br>