[PATCH] D48903: [VirtualFileSystem] InMemoryFileSystem::status: Return a Status with the requested name

Simon Marchi via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Aug 1 10:52:35 PDT 2018

simark added a subscriber: eric_niebler.
simark added a comment.

@eric_niebler, question for you:

This patch causes clang to emit a `-Wnonportable-include-path` warning where it did not before.  It affects the following test on Windows:

The warning is currently not emitted for the **last** clang invocation (the one that includes PCH), because the real path value is not available, and therefore this condition is false:

With this patch, the real path value is available, so we emit the warning.  Is it on purpose that this warning is not emitted in this case?  If not should I simply add `-Wno-nonportable-include-path` to the last clang invocation, as was done earlier with the first invocation?

It seems to me like the warning is valid, even though we use precompiled headers.

  rC Clang


More information about the cfe-commits mailing list