diff options
| author | Volodymyr Sapsai <vsapsai@apple.com> | 2018-10-24 22:39:38 +0000 |
|---|---|---|
| committer | Volodymyr Sapsai <vsapsai@apple.com> | 2018-10-24 22:39:38 +0000 |
| commit | ee89b2e01dfb29ceb58748f9a8ad173430a01d18 (patch) | |
| tree | 8850ac78d4c7b57a8f0e5b1a1bf2841518ac3437 /clang/test/Modules/crash-vfs-headermaps.m | |
| parent | 5e707cb274911a76e129c1faa71dd95d2b580101 (diff) | |
| download | bcm5719-llvm-ee89b2e01dfb29ceb58748f9a8ad173430a01d18.tar.gz bcm5719-llvm-ee89b2e01dfb29ceb58748f9a8ad173430a01d18.zip | |
[VFS] Remove 'ignore-non-existent-contents' attribute for YAML-based VFS.
'ignore-non-existent-contents' stopped working after r342232 in a way
that the actual attribute value isn't used and it works as if it is
always `true`.
Common use case for VFS iteration is iterating through files in umbrella
directories for modules. Ability to detect if some VFS entries point to
non-existing files is nice but non-critical. Instead of adding back
support for `'ignore-non-existent-contents': false` I am removing the
attribute, because such scenario isn't used widely enough and stricter
checks don't provide enough value to justify the maintenance.
rdar://problem/45176119
Reviewers: bruno
Reviewed By: bruno
Subscribers: hiraditya, dexonsmith, sammccall, cfe-commits
Differential Revision: https://reviews.llvm.org/D53228
llvm-svn: 345212
Diffstat (limited to 'clang/test/Modules/crash-vfs-headermaps.m')
| -rw-r--r-- | clang/test/Modules/crash-vfs-headermaps.m | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/clang/test/Modules/crash-vfs-headermaps.m b/clang/test/Modules/crash-vfs-headermaps.m index 118537ba2a4..f4c101cd0c6 100644 --- a/clang/test/Modules/crash-vfs-headermaps.m +++ b/clang/test/Modules/crash-vfs-headermaps.m @@ -32,7 +32,6 @@ // CHECKYAML: 'case-sensitive': // CHECKYAML-NEXT: 'use-external-names': 'false', // CHECKYAML-NEXT: 'overlay-relative': 'true', -// CHECKYAML-NEXT: 'ignore-non-existent-contents': 'false' // CHECKYAML: 'type': 'directory' // CHECKYAML: 'name': "/[[PATH:.*]]/Foo.framework/Headers", // CHECKYAML-NEXT: 'contents': [ |
