Re: [PATCH] D23422: [VFS] Add 'ignore-non-existent-contents' field to YAML

2016-08-11 Thread Bruno Cardoso Lopes via cfe-commits
bruno accepted this revision. bruno added a reviewer: bruno. bruno added a comment. This revision is now accepted and ready to land. Applied your comments, and committed in r278456! Thanks https://reviews.llvm.org/D23422 ___ cfe-commits mailing lis

Re: [PATCH] D23422: [VFS] Add 'ignore-non-existent-contents' field to YAML

2016-08-11 Thread Ben Langmuir via cfe-commits
benlangmuir added a comment. This should have a test for the writer change. Otherwise LGTM. https://reviews.llvm.org/D23422 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D23422: [VFS] Add 'ignore-non-existent-contents' field to YAML

2016-08-11 Thread Bruno Cardoso Lopes via cfe-commits
bruno created this revision. bruno added a reviewer: benlangmuir. bruno added a subscriber: cfe-commits. Add 'ignore-non-existent-contents' to tell the VFS whether an invalid path obtained via 'external-contents' should cause iteration on the VFS to stop. If 'true', the VFS should ignore the entr