diff --git a/clang/lib/Basic/VirtualFileSystem.cpp b/clang/lib/Basic/VirtualFileSystem.cpp index 62a386b73528..a56c9115fb1b 100644 --- a/clang/lib/Basic/VirtualFileSystem.cpp +++ b/clang/lib/Basic/VirtualFileSystem.cpp @@ -1778,29 +1778,47 @@ VFSFromYamlDirIterImpl::VFSFromYamlDirIterImpl( RedirectingDirectoryEntry::iterator Begin, RedirectingDirectoryEntry::iterator End, std::error_code &EC) : Dir(_Path.str()), FS(FS), Current(Begin), End(End) { - if (Current != End) { + while (Current != End) { SmallString<128> PathStr(Dir); llvm::sys::path::append(PathStr, (*Current)->getName()); llvm::ErrorOr S = FS.status(PathStr); - if (S) + if (S) { CurrentEntry = *S; - else + return; + } + // Skip entries which do not map to a reliable external content. + if (FS.ignoreNonExistentContents() && + S.getError() == llvm::errc::no_such_file_or_directory) { + ++Current; + continue; + } else { EC = S.getError(); + break; + } } } std::error_code VFSFromYamlDirIterImpl::increment() { assert(Current != End && "cannot iterate past end"); - if (++Current != End) { + while (++Current != End) { SmallString<128> PathStr(Dir); llvm::sys::path::append(PathStr, (*Current)->getName()); llvm::ErrorOr S = FS.status(PathStr); - if (!S) - return S.getError(); + if (!S) { + // Skip entries which do not map to a reliable external content. + if (FS.ignoreNonExistentContents() && + S.getError() == llvm::errc::no_such_file_or_directory) { + continue; + } else { + return S.getError(); + } + } CurrentEntry = *S; - } else { - CurrentEntry = Status(); + break; } + + if (Current == End) + CurrentEntry = Status(); return std::error_code(); } diff --git a/clang/test/VFS/Inputs/Bar.framework/Headers/A.h b/clang/test/VFS/Inputs/Bar.framework/Headers/A.h new file mode 100644 index 000000000000..975f1f0437bb --- /dev/null +++ b/clang/test/VFS/Inputs/Bar.framework/Headers/A.h @@ -0,0 +1 @@ +// A.h diff --git a/clang/test/VFS/Inputs/Bar.framework/Headers/B.h b/clang/test/VFS/Inputs/Bar.framework/Headers/B.h new file mode 100644 index 000000000000..761540b09cb3 --- /dev/null +++ b/clang/test/VFS/Inputs/Bar.framework/Headers/B.h @@ -0,0 +1 @@ +// B.h diff --git a/clang/test/VFS/Inputs/Bar.framework/Headers/C.h b/clang/test/VFS/Inputs/Bar.framework/Headers/C.h new file mode 100644 index 000000000000..a0121d46b41f --- /dev/null +++ b/clang/test/VFS/Inputs/Bar.framework/Headers/C.h @@ -0,0 +1 @@ +// C.h diff --git a/clang/test/VFS/Inputs/Bar.framework/Modules/module.modulemap b/clang/test/VFS/Inputs/Bar.framework/Modules/module.modulemap new file mode 100644 index 000000000000..d5c759aceb74 --- /dev/null +++ b/clang/test/VFS/Inputs/Bar.framework/Modules/module.modulemap @@ -0,0 +1,6 @@ +framework module Bar [extern_c] { + umbrella "Headers" + export * + module * { export * } +} + diff --git a/clang/test/VFS/Inputs/bar-headers.yaml b/clang/test/VFS/Inputs/bar-headers.yaml new file mode 100644 index 000000000000..846d55cc9ece --- /dev/null +++ b/clang/test/VFS/Inputs/bar-headers.yaml @@ -0,0 +1,39 @@ +{ + 'version': 0, + 'case-sensitive': 'false', + 'ignore-non-existent-contents': 'true', + 'roots': [ + { + 'type': 'directory', + 'name': "VDIR/Bar.framework/Headers", + 'contents': [ + { + 'type': 'file', + 'name': "A.h", + 'external-contents': "OUT_DIR/Bar.framework/Headers/A.h" + }, + { + 'type': 'file', + 'name': "B.h", + 'external-contents': "OUT_DIR/Bar.framework/Headers/B.h" + }, + { + 'type': 'file', + 'name': "C.h", + 'external-contents': "OUT_DIR/Bar.framework/Headers/C.h" + } + ] + }, + { + 'type': 'directory', + 'name': "VDIR/Bar.framework/Modules", + 'contents': [ + { + 'type': 'file', + 'name': "module.modulemap", + 'external-contents': "OUT_DIR/Bar.framework/Modules/module.modulemap" + } + ] + }, + ] +} diff --git a/clang/test/VFS/umbrella-framework-import-skipnonexist.m b/clang/test/VFS/umbrella-framework-import-skipnonexist.m new file mode 100644 index 000000000000..5c7cd6d9db13 --- /dev/null +++ b/clang/test/VFS/umbrella-framework-import-skipnonexist.m @@ -0,0 +1,14 @@ +// REQUIRES: crash-recovery, shell + +// FIXME: This XFAIL is cargo-culted from crash-report.c. Do we need it? +// XFAIL: mingw32 + +// RUN: rm -rf %t +// RUN: mkdir -p %t/vdir %t/outdir %t/cache +// RUN: cp -a %S/Inputs/Bar.framework %t/outdir/ +// +// RUN: sed -e "s:VDIR:%t/vdir:g" -e "s:OUT_DIR:%t/outdir:g" %S/Inputs/bar-headers.yaml > %t/vdir/bar-headers.yaml +// RUN: rm -f %t/outdir/Bar.framework/Headers/B.h +// RUN: %clang_cc1 -fmodules -fimplicit-module-maps -fmodules-cache-path=%t/cache -ivfsoverlay %t/vdir/bar-headers.yaml -F %t/vdir -fsyntax-only %s + +@import Bar;