[cfe-commits] [win32][review request] clang Index test fix
NAKAMURA Takumi
geek4civic at gmail.com
Mon Oct 4 02:01:43 PDT 2010
Francois,
On mingw, your patch improves test/Index/remap-load.c. Thanks!
test/Index/remap-complete works regardless of your patch.
I don't know why :p
FYI ...Takumi
2010/10/4 Francois Pichet <pichet2000 at gmail.com>:
> This patch remove the Win32 XFAIL from test/Index/remap-complete.c and
> test/Index/remap-load.c
>
> The problem was that text files were open in text mode and Microsoft
> implementation of fread and write will try to do nasty line-feed
> conversion which make the line position no longer valid.
>
> I think the option llvm::raw_fd_ostream::F_Binary should always be
> used. Or it should be removed and always applied. Or else we are going
> to have more problem with that.
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>
>
More information about the cfe-commits
mailing list