Skip to content

Commit 409b4b5

Browse files
JDevlieghereJDevlieghere
committed
Revert "[VirtualFileSystem] Support virtual working directory in the RedirectingFS"
This reverts the original commit and the follow up: Revert "[VirtualFileSystem] Support virtual working directory in the RedirectingFS" Revert "[test] Update YAML mapping in VirtualFileSystemTest" llvm-svn: 374935
1 parent f14642f commit 409b4b5

File tree

3 files changed

+16
-217
lines changed

3 files changed

+16
-217
lines changed

‎llvm/include/llvm/Support/VirtualFileSystem.h

Lines changed: 2 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -647,19 +647,9 @@ class RedirectingFileSystem : public vfs::FileSystem {
647647
friend class VFSFromYamlDirIterImpl;
648648
friend class RedirectingFileSystemParser;
649649

650-
bool shouldUseExternalFS() const {
651-
return ExternalFSValidWD && IsFallthrough;
652-
}
653-
654650
/// The root(s) of the virtual file system.
655651
std::vector<std::unique_ptr<Entry>> Roots;
656652

657-
/// The current working directory of the file system.
658-
std::string WorkingDirectory;
659-
660-
/// Whether the current working directory is valid for the external FS.
661-
bool ExternalFSValidWD = false;
662-
663653
/// The file system to use for external references.
664654
IntrusiveRefCntPtr<FileSystem> ExternalFS;
665655

@@ -699,7 +689,8 @@ class RedirectingFileSystem : public vfs::FileSystem {
699689
true;
700690
#endif
701691

702-
RedirectingFileSystem(IntrusiveRefCntPtr<FileSystem> ExternalFS);
692+
RedirectingFileSystem(IntrusiveRefCntPtr<FileSystem> ExternalFS)
693+
: ExternalFS(std::move(ExternalFS)) {}
703694

704695
/// Looks up the path <tt>[Start, End)</tt> in \p From, possibly
705696
/// recursing into the contents of \p From if it is a directory.

‎llvm/lib/Support/VirtualFileSystem.cpp

Lines changed: 10 additions & 35 deletions
Original file line numberDiff line numberDiff line change
@@ -989,16 +989,6 @@ std::error_code InMemoryFileSystem::isLocal(const Twine &Path, bool &Result) {
989989
// RedirectingFileSystem implementation
990990
//===-----------------------------------------------------------------------===/
991991

992-
RedirectingFileSystem::RedirectingFileSystem(IntrusiveRefCntPtr<FileSystem> FS)
993-
: ExternalFS(std::move(FS)) {
994-
if (ExternalFS)
995-
if (auto ExternalWorkingDirectory =
996-
ExternalFS->getCurrentWorkingDirectory()) {
997-
WorkingDirectory = *ExternalWorkingDirectory;
998-
ExternalFSValidWD = true;
999-
}
1000-
}
1001-
1002992
// FIXME: reuse implementation common with OverlayFSDirIterImpl as these
1003993
// iterators are conceptually similar.
1004994
class llvm::vfs::VFSFromYamlDirIterImpl
@@ -1045,27 +1035,12 @@ class llvm::vfs::VFSFromYamlDirIterImpl
10451035

10461036
llvm::ErrorOr<std::string>
10471037
RedirectingFileSystem::getCurrentWorkingDirectory() const {
1048-
return WorkingDirectory;
1038+
return ExternalFS->getCurrentWorkingDirectory();
10491039
}
10501040

10511041
std::error_code
10521042
RedirectingFileSystem::setCurrentWorkingDirectory(const Twine &Path) {
1053-
// Don't change the working directory if the path doesn't exist.
1054-
if (!exists(Path))
1055-
return errc::no_such_file_or_directory;
1056-
1057-
// Always change the external FS but ignore its result.
1058-
if (ExternalFS) {
1059-
auto EC = ExternalFS->setCurrentWorkingDirectory(Path);
1060-
ExternalFSValidWD = !static_cast<bool>(EC);
1061-
}
1062-
1063-
SmallString<128> AbsolutePath;
1064-
Path.toVector(AbsolutePath);
1065-
if (std::error_code EC = makeAbsolute(AbsolutePath))
1066-
return EC;
1067-
WorkingDirectory = AbsolutePath.str();
1068-
return {};
1043+
return ExternalFS->setCurrentWorkingDirectory(Path);
10691044
}
10701045

10711046
std::error_code RedirectingFileSystem::isLocal(const Twine &Path,
@@ -1078,7 +1053,7 @@ directory_iterator RedirectingFileSystem::dir_begin(const Twine &Dir,
10781053
ErrorOr<RedirectingFileSystem::Entry *> E = lookupPath(Dir);
10791054
if (!E) {
10801055
EC = E.getError();
1081-
if (shouldUseExternalFS() && EC == errc::no_such_file_or_directory)
1056+
if (IsFallthrough && EC == errc::no_such_file_or_directory)
10821057
return ExternalFS->dir_begin(Dir, EC);
10831058
return {};
10841059
}
@@ -1096,7 +1071,7 @@ directory_iterator RedirectingFileSystem::dir_begin(const Twine &Dir,
10961071
auto *D = cast<RedirectingFileSystem::RedirectingDirectoryEntry>(*E);
10971072
return directory_iterator(std::make_shared<VFSFromYamlDirIterImpl>(
10981073
Dir, D->contents_begin(), D->contents_end(),
1099-
/*IterateExternalFS=*/shouldUseExternalFS(), *ExternalFS, EC));
1074+
/*IterateExternalFS=*/IsFallthrough, *ExternalFS, EC));
11001075
}
11011076

11021077
void RedirectingFileSystem::setExternalContentsPrefixDir(StringRef PrefixDir) {
@@ -1597,7 +1572,7 @@ RedirectingFileSystem::create(std::unique_ptr<MemoryBuffer> Buffer,
15971572
RedirectingFileSystemParser P(Stream);
15981573

15991574
std::unique_ptr<RedirectingFileSystem> FS(
1600-
new RedirectingFileSystem(ExternalFS));
1575+
new RedirectingFileSystem(std::move(ExternalFS)));
16011576

16021577
if (!YAMLFilePath.empty()) {
16031578
// Use the YAML path from -ivfsoverlay to compute the dir to be prefixed
@@ -1726,7 +1701,7 @@ ErrorOr<Status> RedirectingFileSystem::status(const Twine &Path,
17261701
ErrorOr<Status> RedirectingFileSystem::status(const Twine &Path) {
17271702
ErrorOr<RedirectingFileSystem::Entry *> Result = lookupPath(Path);
17281703
if (!Result) {
1729-
if (shouldUseExternalFS() &&
1704+
if (IsFallthrough &&
17301705
Result.getError() == llvm::errc::no_such_file_or_directory) {
17311706
return ExternalFS->status(Path);
17321707
}
@@ -1764,7 +1739,7 @@ ErrorOr<std::unique_ptr<File>>
17641739
RedirectingFileSystem::openFileForRead(const Twine &Path) {
17651740
ErrorOr<RedirectingFileSystem::Entry *> E = lookupPath(Path);
17661741
if (!E) {
1767-
if (shouldUseExternalFS() &
1742+
if (IsFallthrough &&
17681743
E.getError() == llvm::errc::no_such_file_or_directory) {
17691744
return ExternalFS->openFileForRead(Path);
17701745
}
@@ -1795,7 +1770,7 @@ RedirectingFileSystem::getRealPath(const Twine &Path,
17951770
SmallVectorImpl<char> &Output) const {
17961771
ErrorOr<RedirectingFileSystem::Entry *> Result = lookupPath(Path);
17971772
if (!Result) {
1798-
if (shouldUseExternalFS() &&
1773+
if (IsFallthrough &&
17991774
Result.getError() == llvm::errc::no_such_file_or_directory) {
18001775
return ExternalFS->getRealPath(Path, Output);
18011776
}
@@ -1808,8 +1783,8 @@ RedirectingFileSystem::getRealPath(const Twine &Path,
18081783
}
18091784
// Even if there is a directory entry, fall back to ExternalFS if allowed,
18101785
// because directories don't have a single external contents path.
1811-
return shouldUseExternalFS() ? ExternalFS->getRealPath(Path, Output)
1812-
: llvm::errc::invalid_argument;
1786+
return IsFallthrough ? ExternalFS->getRealPath(Path, Output)
1787+
: llvm::errc::invalid_argument;
18131788
}
18141789

18151790
IntrusiveRefCntPtr<FileSystem>

‎llvm/unittests/Support/VirtualFileSystemTest.cpp

Lines changed: 4 additions & 171 deletions
Original file line numberDiff line numberDiff line change
@@ -41,9 +41,7 @@ struct DummyFile : public vfs::File {
4141
class DummyFileSystem : public vfs::FileSystem {
4242
int FSID; // used to produce UniqueIDs
4343
int FileID; // used to produce UniqueIDs
44-
std::string WorkingDirectory;
4544
std::map<std::string, vfs::Status> FilesAndDirs;
46-
typedef std::map<std::string, vfs::Status>::const_iterator const_iterator;
4745

4846
static int getNextFSID() {
4947
static int Count = 0;
@@ -54,7 +52,8 @@ class DummyFileSystem : public vfs::FileSystem {
5452
DummyFileSystem() : FSID(getNextFSID()), FileID(0) {}
5553

5654
ErrorOr<vfs::Status> status(const Twine &Path) override {
57-
auto I = findEntry(Path);
55+
std::map<std::string, vfs::Status>::iterator I =
56+
FilesAndDirs.find(Path.str());
5857
if (I == FilesAndDirs.end())
5958
return make_error_code(llvm::errc::no_such_file_or_directory);
6059
return I->second;
@@ -67,16 +66,15 @@ class DummyFileSystem : public vfs::FileSystem {
6766
return S.getError();
6867
}
6968
llvm::ErrorOr<std::string> getCurrentWorkingDirectory() const override {
70-
return WorkingDirectory;
69+
return std::string();
7170
}
7271
std::error_code setCurrentWorkingDirectory(const Twine &Path) override {
73-
WorkingDirectory = Path.str();
7472
return std::error_code();
7573
}
7674
// Map any symlink to "/symlink".
7775
std::error_code getRealPath(const Twine &Path,
7876
SmallVectorImpl<char> &Output) const override {
79-
auto I = findEntry(Path);
77+
auto I = FilesAndDirs.find(Path.str());
8078
if (I == FilesAndDirs.end())
8179
return make_error_code(llvm::errc::no_such_file_or_directory);
8280
if (I->second.isSymlink()) {
@@ -138,14 +136,6 @@ class DummyFileSystem : public vfs::FileSystem {
138136
FilesAndDirs[Path] = Status;
139137
}
140138

141-
const_iterator findEntry(const Twine &Path) const {
142-
SmallString<128> P;
143-
Path.toVector(P);
144-
std::error_code EC = makeAbsolute(P);
145-
assert(!EC);
146-
return FilesAndDirs.find(P.str());
147-
}
148-
149139
void addRegularFile(StringRef Path, sys::fs::perms Perms = sys::fs::all_all) {
150140
vfs::Status S(Path, UniqueID(FSID, FileID++),
151141
std::chrono::system_clock::now(), 0, 0, 1024,
@@ -168,12 +158,6 @@ class DummyFileSystem : public vfs::FileSystem {
168158
}
169159
};
170160

171-
class ErrorDummyFileSystem : public DummyFileSystem {
172-
std::error_code setCurrentWorkingDirectory(const Twine &Path) override {
173-
return llvm::errc::no_such_file_or_directory;
174-
}
175-
};
176-
177161
/// Replace back-slashes by front-slashes.
178162
std::string getPosixPath(std::string S) {
179163
SmallString<128> Result;
@@ -2010,154 +1994,3 @@ TEST_F(VFSFromYAMLTest, GetRealPath) {
20101994
EXPECT_EQ(FS->getRealPath("/non_existing", RealPath),
20111995
errc::no_such_file_or_directory);
20121996
}
2013-
2014-
TEST_F(VFSFromYAMLTest, WorkingDirectory) {
2015-
IntrusiveRefCntPtr<DummyFileSystem> Lower(new DummyFileSystem());
2016-
Lower->addDirectory("//root/");
2017-
Lower->addDirectory("//root/foo");
2018-
Lower->addRegularFile("//root/foo/a");
2019-
Lower->addRegularFile("//root/foo/b");
2020-
IntrusiveRefCntPtr<vfs::FileSystem> FS = getFromYAMLString(
2021-
"{ 'use-external-names': false,\n"
2022-
" 'roots': [\n"
2023-
"{\n"
2024-
" 'type': 'directory',\n"
2025-
" 'name': '//root/bar',\n"
2026-
" 'contents': [ {\n"
2027-
" 'type': 'file',\n"
2028-
" 'name': 'a',\n"
2029-
" 'external-contents': '//root/foo/a'\n"
2030-
" }\n"
2031-
" ]\n"
2032-
"}\n"
2033-
"]\n"
2034-
"}",
2035-
Lower);
2036-
ASSERT_TRUE(FS.get() != nullptr);
2037-
std::error_code EC = FS->setCurrentWorkingDirectory("//root/bar/");
2038-
ASSERT_FALSE(EC);
2039-
2040-
llvm::ErrorOr<std::string> WorkingDir = FS->getCurrentWorkingDirectory();
2041-
ASSERT_TRUE(WorkingDir);
2042-
EXPECT_EQ(*WorkingDir, "//root/bar/");
2043-
2044-
llvm::ErrorOr<vfs::Status> Status = FS->status("./a");
2045-
ASSERT_FALSE(Status.getError());
2046-
EXPECT_TRUE(Status->isStatusKnown());
2047-
EXPECT_FALSE(Status->isDirectory());
2048-
EXPECT_TRUE(Status->isRegularFile());
2049-
EXPECT_FALSE(Status->isSymlink());
2050-
EXPECT_FALSE(Status->isOther());
2051-
EXPECT_TRUE(Status->exists());
2052-
2053-
EC = FS->setCurrentWorkingDirectory("bogus");
2054-
ASSERT_TRUE(EC);
2055-
WorkingDir = FS->getCurrentWorkingDirectory();
2056-
ASSERT_TRUE(WorkingDir);
2057-
EXPECT_EQ(*WorkingDir, "//root/bar/");
2058-
2059-
EC = FS->setCurrentWorkingDirectory("//root/");
2060-
ASSERT_FALSE(EC);
2061-
WorkingDir = FS->getCurrentWorkingDirectory();
2062-
ASSERT_TRUE(WorkingDir);
2063-
EXPECT_EQ(*WorkingDir, "//root/");
2064-
2065-
EC = FS->setCurrentWorkingDirectory("bar/");
2066-
ASSERT_FALSE(EC);
2067-
WorkingDir = FS->getCurrentWorkingDirectory();
2068-
ASSERT_TRUE(WorkingDir);
2069-
EXPECT_EQ(*WorkingDir, "//root/bar/");
2070-
}
2071-
2072-
TEST_F(VFSFromYAMLTest, WorkingDirectoryFallthrough) {
2073-
IntrusiveRefCntPtr<DummyFileSystem> Lower(new DummyFileSystem());
2074-
Lower->addDirectory("//root/");
2075-
Lower->addDirectory("//root/foo");
2076-
Lower->addRegularFile("//root/foo/a");
2077-
Lower->addRegularFile("//root/foo/b");
2078-
Lower->addRegularFile("//root/c");
2079-
IntrusiveRefCntPtr<vfs::FileSystem> FS = getFromYAMLString(
2080-
"{ 'use-external-names': false,\n"
2081-
" 'roots': [\n"
2082-
"{\n"
2083-
" 'type': 'directory',\n"
2084-
" 'name': '//root/bar',\n"
2085-
" 'contents': [ {\n"
2086-
" 'type': 'file',\n"
2087-
" 'name': 'a',\n"
2088-
" 'external-contents': '//root/foo/a'\n"
2089-
" }\n"
2090-
" ]\n"
2091-
"}\n"
2092-
"]\n"
2093-
"}",
2094-
Lower);
2095-
ASSERT_TRUE(FS.get() != nullptr);
2096-
std::error_code EC = FS->setCurrentWorkingDirectory("//root/");
2097-
ASSERT_FALSE(EC);
2098-
ASSERT_TRUE(FS.get() != nullptr);
2099-
2100-
llvm::ErrorOr<vfs::Status> Status = FS->status("bar/a");
2101-
ASSERT_FALSE(Status.getError());
2102-
EXPECT_TRUE(Status->exists());
2103-
2104-
Status = FS->status("foo/a");
2105-
ASSERT_FALSE(Status.getError());
2106-
EXPECT_TRUE(Status->exists());
2107-
2108-
EC = FS->setCurrentWorkingDirectory("//root/bar/");
2109-
ASSERT_FALSE(EC);
2110-
2111-
Status = FS->status("./a");
2112-
ASSERT_FALSE(Status.getError());
2113-
EXPECT_TRUE(Status->exists());
2114-
2115-
Status = FS->status("./b");
2116-
ASSERT_TRUE(Status.getError());
2117-
2118-
Status = FS->status("./c");
2119-
ASSERT_TRUE(Status.getError());
2120-
2121-
EC = FS->setCurrentWorkingDirectory("//root/");
2122-
ASSERT_FALSE(EC);
2123-
2124-
Status = FS->status("c");
2125-
ASSERT_FALSE(Status.getError());
2126-
EXPECT_TRUE(Status->exists());
2127-
}
2128-
2129-
TEST_F(VFSFromYAMLTest, WorkingDirectoryFallthroughInvalid) {
2130-
IntrusiveRefCntPtr<ErrorDummyFileSystem> Lower(new ErrorDummyFileSystem());
2131-
Lower->addDirectory("//root/");
2132-
Lower->addDirectory("//root/foo");
2133-
Lower->addRegularFile("//root/foo/a");
2134-
Lower->addRegularFile("//root/foo/b");
2135-
Lower->addRegularFile("//root/c");
2136-
IntrusiveRefCntPtr<vfs::FileSystem> FS = getFromYAMLString(
2137-
"{ 'use-external-names': false,\n"
2138-
" 'roots': [\n"
2139-
"{\n"
2140-
" 'type': 'directory',\n"
2141-
" 'name': '//root/bar',\n"
2142-
" 'contents': [ {\n"
2143-
" 'type': 'file',\n"
2144-
" 'name': 'a',\n"
2145-
" 'external-contents': '//root/foo/a'\n"
2146-
" }\n"
2147-
" ]\n"
2148-
"}\n"
2149-
"]\n"
2150-
"}",
2151-
Lower);
2152-
ASSERT_TRUE(FS.get() != nullptr);
2153-
std::error_code EC = FS->setCurrentWorkingDirectory("//root/");
2154-
ASSERT_FALSE(EC);
2155-
ASSERT_TRUE(FS.get() != nullptr);
2156-
2157-
llvm::ErrorOr<vfs::Status> Status = FS->status("bar/a");
2158-
ASSERT_FALSE(Status.getError());
2159-
EXPECT_TRUE(Status->exists());
2160-
2161-
Status = FS->status("foo/a");
2162-
ASSERT_TRUE(Status.getError());
2163-
}

0 commit comments

Comments
 (0)