From: Volodymyr Sapsai Date: Fri, 16 Nov 2018 01:18:04 +0000 (+0000) Subject: [VFS] Implement `RedirectingFileSystem::getRealPath`. X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=30a86941b0515629cdac77d4ef8078422e6fe706;p=clang [VFS] Implement `RedirectingFileSystem::getRealPath`. 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 add a subframework as a submodule. And when we parse module map for the subframework, we would encounter an error like > error: umbrella for module 'WithSubframework.Foo' already covers this directory By implementing `getRealPath` "an egregious but useful hack" in `ModuleMap::inferFrameworkModule` works as expected. LLVM commit is r347009. rdar://problem/45821279 Reviewers: bruno, benlangmuir, erik.pilkington Reviewed By: bruno Subscribers: hiraditya, dexonsmith, JDevlieghere, cfe-commits, llvm-commits Differential Revision: https://reviews.llvm.org/D54245 git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@347012 91177308-0d34-0410-b5e6-96231b3b80d8 --- diff --git a/test/VFS/subframework-symlink.m b/test/VFS/subframework-symlink.m new file mode 100644 index 0000000000..1c6fd06409 --- /dev/null +++ b/test/VFS/subframework-symlink.m @@ -0,0 +1,23 @@ +// REQUIRES: shell + +// Test that when a subframework is a symlink to another framework, we don't +// add it as a submodule to the enclosing framework. We also need to make clang +// to infer module for the enclosing framework. For this we don't have +// a module map for the framework itself but have it in a parent directory. +// +// RUN: rm -rf %t +// RUN: mkdir %t +// RUN: echo 'framework module * {}' > %t/module.modulemap +// RUN: mkdir -p %t/WithSubframework.framework/Headers +// RUN: echo '#include ' > %t/WithSubframework.framework/Headers/WithSubframework.h +// RUN: cp -R %S/Inputs/Foo.framework %t +// RUN: mkdir -p %t/WithSubframework.framework/Frameworks +// RUN: ln -s %t/Foo.framework %t/WithSubframework.framework/Frameworks +// RUN: %clang_cc1 -fmodules -fimplicit-module-maps -fmodules-cache-path=%t/cache1 -F %t -fsyntax-only %s + +// Adding VFS overlay shouldn't change this behavior. +// +// RUN: sed -e "s:INPUT_DIR:/InvalidPath:g" -e "s:OUT_DIR:/InvalidPath:g" %S/Inputs/vfsoverlay.yaml > %t/overlay.yaml +// RUN: %clang_cc1 -fmodules -fimplicit-module-maps -fmodules-cache-path=%t/cache2 -F %t -fsyntax-only %s -ivfsoverlay %t/overlay.yaml + +#import