[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