[PATCH] D54245: [VFS] Implement `RedirectingFileSystem::getRealPath`.

2018-11-15 Thread Volodymyr Sapsai via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL347009: [VFS] Implement `RedirectingFileSystem::getRealPath`. (authored by vsapsai, committed by ). Herald added a subscriber: kristina. Changed prior to commit:

[PATCH] D54245: [VFS] Implement `RedirectingFileSystem::getRealPath`.

2018-11-15 Thread Bruno Cardoso Lopes via Phabricator via cfe-commits
bruno accepted this revision. bruno added a comment. This revision is now accepted and ready to land. Herald added a subscriber: jkorous. LGTM https://reviews.llvm.org/D54245 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D54245: [VFS] Implement `RedirectingFileSystem::getRealPath`.

2018-11-13 Thread Volodymyr Sapsai via Phabricator via cfe-commits
vsapsai added a comment. Ping. https://reviews.llvm.org/D54245 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D54245: [VFS] Implement `RedirectingFileSystem::getRealPath`.

2018-11-08 Thread Volodymyr Sapsai via Phabricator via cfe-commits
vsapsai added a comment. In https://reviews.llvm.org/D54245#1292008, @bruno wrote: > Hi Volodymyr. Does this behavior changed after your VFS fallback change? https://reviews.llvm.org/D50539 didn't change `RedirectingFileSystem` behavior regarding real paths. But for `-ivfsoverlay` it started

[PATCH] D54245: [VFS] Implement `RedirectingFileSystem::getRealPath`.

2018-11-08 Thread Bruno Cardoso Lopes via Phabricator via cfe-commits
bruno added a comment. Hi Volodymyr. Does this behavior changed after your VFS fallback change? https://reviews.llvm.org/D54245 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D54245: [VFS] Implement `RedirectingFileSystem::getRealPath`.

2018-11-07 Thread Volodymyr Sapsai via Phabricator via cfe-commits
vsapsai created this revision. vsapsai added reviewers: bruno, benlangmuir. Herald added subscribers: dexonsmith, hiraditya. It fixes the case when Objective-C framework is added as a subframework through a symlink. When parent framework infers a module map and fails to detect a symlink, it would