[PATCH] D11944: Nativize filename in FileManager::getFile().

Manuel Klimek via cfe-commits cfe-commits at lists.llvm.org
Thu Aug 13 05:01:21 PDT 2015


klimek added a comment.

In http://reviews.llvm.org/D11944#223597, @chapuni wrote:

> I wish, in clang/llvm, internal path separator would be slash.
>  It'd be better to use backslash where it'd be really required, for example interface to cmd.exe or MS toolchain.
>
> For me, native-ization is nightmare.


Are you saying that all MS functions we access support / as path separator on the platforms we support, or that we should canonicalize all filenames to / when we first get them, and convert them to \-based when we hit the MS file APIs?


http://reviews.llvm.org/D11944





More information about the cfe-commits mailing list