<div dir="ltr">Given that r203614 is in and this patch is the same kind of thing, I retract my being opposed to this patch.</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Mar 6, 2014 at 2:29 PM, David Majnemer <span dir="ltr"><<a href="mailto:david.majnemer@gmail.com" target="_blank">david.majnemer@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Modulo needing some style fixes, this looks fine. However, I'd prefer for @rsmith look at this at least to make sure it plays nice with modules.<div class="HOEnZb">
<div class="h5"><br><br><div>On Wed Mar 05 2014 at 1:18:22 PM, Nico Weber <<a href="mailto:thakis@chromium.org" target="_blank">thakis@chromium.org</a>> wrote:</div>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">The compiler won't be the only place with issues like this though, no? Say you have a generated header "header.h" and some file accidentally includes it as "Header.h", then the build system now won't have a correct dependency edge. There's probably more examples.<br>


<br>Can't you just create a case-insensitive file system even in cross mode?</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Mar 5, 2014 at 1:13 PM, Saleem Abdulrasool <span dir="ltr"><<a href="mailto:abdulras@fb.com" target="_blank">abdulras@fb.com</a>></span> wrote:<br>


<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
  One of the nice things about clang is that it is inherently a cross-compiler :-).  Not all file systems are case-insensitive, and unfortunately, as I mentioned in the commit message, the Windows SDK does get the case wrong in many cases.  This is an attempt to make the cross-compiler work properly with MS compatibility irrespective of the environment where it is being executed.<br>



<br>
<a href="http://llvm-reviews.chandlerc.com/D2972" target="_blank">http://llvm-reviews.chandlerc.com/D2972</a><br>
</blockquote></div><br></div>
</blockquote>
</div></div></blockquote></div><br></div>