[PATCH] Fix for include file resolution on Windows

Yaron Keren yaron.keren at gmail.com
Thu Jan 8 05:59:45 PST 2015


Fixing the test should be easy, the problem is deeper, FileManager may have
two different entries for the same file since it does not know c:\my\file
is same as c:/my/file, wouldn't that cause problems in clang operation?




2015-01-08 15:32 GMT+02:00 Rafael Ávila de Espíndola <
rafael.espindola at gmail.com>:

> It should be possible to add a testcase showing that we now add a \
> instead of a / on windows, no?
>
>
> http://reviews.llvm.org/D6871
>
> EMAIL PREFERENCES
>   http://reviews.llvm.org/settings/panel/emailpreferences/
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20150108/76bdac6f/attachment.html>


More information about the cfe-commits mailing list