<div dir="rtl"><div dir="ltr"><span style="font-family:arial,sans-serif;font-size:12.800000190734863px">Lexer/cross-windows-on-linux-</span><span style="font-family:arial,sans-serif;font-size:12.800000190734863px">default.cpp is marked</span><br>

</div><div dir="ltr"><span style="font-family:arial,sans-serif;font-size:12.800000190734863px"><br></span></div><div dir="ltr"><font face="arial, sans-serif"> REQUIRES: disabled</font><br></div><div dir="ltr"><font face="arial, sans-serif"><br>

</font></div><div dir="ltr"><font face="arial, sans-serif">doesn't it means the test is disabled?</font></div><div dir="ltr"><font face="arial, sans-serif"><br></font></div><div dir="ltr"><font face="arial, sans-serif">I had not debugged the six tests to see what exactly broke, only restored pre-patch behaviour to make the tests pass.</font></div>

<div dir="ltr"><font face="arial, sans-serif"><br></font></div><div dir="ltr"><font face="arial, sans-serif">On Windows both / and \ are accepted.  From past experience ( and <a href="http://reviews.llvm.org/D3686">http://reviews.llvm.org/D3686</a> )</font></div>

<div dir="ltr"><font face="arial, sans-serif">such problems are usually caused by StringMap caches in which /  and \ are considered different when searching. Prime suspects would be:</font></div><div dir="ltr"><font face="arial, sans-serif"><br>

</font></div><div dir="ltr"><font face="arial, sans-serif"> HeaderSearch::LookupFileCache<br></font></div><div dir="ltr"><span style="font-family:arial,sans-serif"> HeaderSearch::</span><font face="arial, sans-serif">FrameworkMap<br>

</font></div><div dir="ltr"><span style="font-family:arial,sans-serif"> FileManager::SeenFileEntries</span><br></div><div dir="ltr"><br></div><div dir="ltr">but these would also need fixing:</div><div dir="ltr"><br></div>

<div dir="ltr"><span style="font-family:arial,sans-serif"> HeaderSearch::</span><font face="arial, sans-serif">IncludeAliases</font><br></div><div dir="ltr"><span style="font-family:arial,sans-serif"> FileManager::</span>SeenDirEntries<br>

</div><div dir="ltr"><br></div><div dir="ltr">and probably other StringMaps.</div><div dir="ltr"><br></div><div dir="ltr">To solve these we could create a PathStringMap derived from StringMap in which the search is  / \ tolerant and use it instead of the StringMap when paths are stored.</div>

<div dir="ltr"><br></div><div dir="ltr">Yaron</div><div dir="ltr"><br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote"><div dir="ltr">2014-08-11 18:46 GMT+03:00 Rafael Espíndola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>></span>:</div>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">> +#ifndef LLVM_ON_WIN32<br>
>      llvm::sys::path::native(NormalizedPath);<br>
> +#endif<br>
<br>
This call is used on linux to make<br>
Lexer/cross-windows-on-linux-default.cpp pass. Since on windows both /<br>
and \ are supposed to work, it seems that LookupFile should succeed<br>
even with the call to llvm::sys::path::native, no? Do you know why it<br>
is failing?<br>
<br>
Cheers,<br>
Rafael<br>
</blockquote></div><br></div>