[PATCH] D114206: [Clang][ScanDeps] Use the virtual path for module maps
Michael Spencer via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Dec 1 16:35:29 PST 2021
Bigcheese added a comment.
There were two issues for Windows.
The first is an actual bug in the Clang VFS that should be fixed separately. You cannot pass just `main.m` as the input to Clang. Clang asks for its parent directory textually, which is `.`. It then tries to stat `.`, which the VFS rejects. Not sure why it's OK on non-Windows.
The other issue is not handling path separators in the FileCheck test. Fixed by adding `sed 's:\\\\\?:/:g'`.
I'm uploading a new patch and will commit on clean precommit tests.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D114206/new/
https://reviews.llvm.org/D114206
More information about the cfe-commits
mailing list