diff options
author | Keith Smiley <keithbsmiley@gmail.com> | 2021-11-13 10:11:25 -0800 |
---|---|---|
committer | Keith Smiley <keithbsmiley@gmail.com> | 2021-11-13 10:11:51 -0800 |
commit | f0cf544d6f6fe6cbca4c07772998272d6bb433d8 (patch) | |
tree | 001b69717097df2c8a0939a7d49d19ea2c0f120b /llvm/unittests/Support/VirtualFileSystemTest.cpp | |
parent | b19e823ff9de52f08bb2d35b91305b3c2d3dd0be (diff) | |
download | llvm-f0cf544d6f6fe6cbca4c07772998272d6bb433d8.zip llvm-f0cf544d6f6fe6cbca4c07772998272d6bb433d8.tar.gz llvm-f0cf544d6f6fe6cbca4c07772998272d6bb433d8.tar.bz2 |
Revert "[VFS] Use original path when falling back to external FS"
```
/work/omp-vega20-0/openmp-offload-amdgpu-runtime/llvm.src/llvm/lib/Support/VirtualFileSystem.cpp: In static member function 'static llvm::ErrorOr<std::unique_ptr<llvm::vfs::File> > llvm::vfs::File::getWithPath(llvm::ErrorOr<std::unique_ptr<llvm::vfs::File> >, const llvm::Twine&)':
/work/omp-vega20-0/openmp-offload-amdgpu-runtime/llvm.src/llvm/lib/Support/VirtualFileSystem.cpp:2084:10: error: could not convert 'F' from 'std::unique_ptr<llvm::vfs::File>' to 'llvm::ErrorOr<std::unique_ptr<llvm::vfs::File> >'
return F;
^
```
This reverts commit c972175649f4bb50d40d911659a04d5620ce6fe0.
Diffstat (limited to 'llvm/unittests/Support/VirtualFileSystemTest.cpp')
-rw-r--r-- | llvm/unittests/Support/VirtualFileSystemTest.cpp | 108 |
1 files changed, 0 insertions, 108 deletions
diff --git a/llvm/unittests/Support/VirtualFileSystemTest.cpp b/llvm/unittests/Support/VirtualFileSystemTest.cpp index be32971..ca35b5e 100644 --- a/llvm/unittests/Support/VirtualFileSystemTest.cpp +++ b/llvm/unittests/Support/VirtualFileSystemTest.cpp @@ -1627,114 +1627,6 @@ TEST_F(VFSFromYAMLTest, RemappedDirectoryOverlayNoFallthrough) { EXPECT_EQ(0, NumDiagnostics); } -TEST_F(VFSFromYAMLTest, ReturnsRequestedPathVFSMiss) { - IntrusiveRefCntPtr<vfs::InMemoryFileSystem> BaseFS( - new vfs::InMemoryFileSystem); - BaseFS->addFile("//root/foo/a", 0, - MemoryBuffer::getMemBuffer("contents of a")); - ASSERT_FALSE(BaseFS->setCurrentWorkingDirectory("//root/foo")); - auto RemappedFS = vfs::RedirectingFileSystem::create( - {}, /*UseExternalNames=*/false, *BaseFS); - - auto OpenedF = RemappedFS->openFileForRead("a"); - ASSERT_FALSE(OpenedF.getError()); - llvm::ErrorOr<std::string> Name = (*OpenedF)->getName(); - ASSERT_FALSE(Name.getError()); - EXPECT_EQ("a", Name.get()); - - auto OpenedS = (*OpenedF)->status(); - ASSERT_FALSE(OpenedS.getError()); - EXPECT_EQ("a", OpenedS->getName()); - EXPECT_FALSE(OpenedS->IsVFSMapped); - - auto DirectS = RemappedFS->status("a"); - ASSERT_FALSE(DirectS.getError()); - EXPECT_EQ("a", DirectS->getName()); - EXPECT_FALSE(DirectS->IsVFSMapped); - - EXPECT_EQ(0, NumDiagnostics); -} - -TEST_F(VFSFromYAMLTest, ReturnsExternalPathVFSHit) { - IntrusiveRefCntPtr<vfs::InMemoryFileSystem> BaseFS( - new vfs::InMemoryFileSystem); - BaseFS->addFile("//root/foo/realname", 0, - MemoryBuffer::getMemBuffer("contents of a")); - auto FS = - getFromYAMLString("{ 'use-external-names': true,\n" - " 'roots': [\n" - "{\n" - " 'type': 'directory',\n" - " 'name': '//root/foo',\n" - " 'contents': [ {\n" - " 'type': 'file',\n" - " 'name': 'vfsname',\n" - " 'external-contents': 'realname'\n" - " }\n" - " ]\n" - "}]}", - BaseFS); - ASSERT_FALSE(FS->setCurrentWorkingDirectory("//root/foo")); - - auto OpenedF = FS->openFileForRead("vfsname"); - ASSERT_FALSE(OpenedF.getError()); - llvm::ErrorOr<std::string> Name = (*OpenedF)->getName(); - ASSERT_FALSE(Name.getError()); - EXPECT_EQ("realname", Name.get()); - - auto OpenedS = (*OpenedF)->status(); - ASSERT_FALSE(OpenedS.getError()); - EXPECT_EQ("realname", OpenedS->getName()); - EXPECT_TRUE(OpenedS->IsVFSMapped); - - auto DirectS = FS->status("vfsname"); - ASSERT_FALSE(DirectS.getError()); - EXPECT_EQ("realname", DirectS->getName()); - EXPECT_TRUE(DirectS->IsVFSMapped); - - EXPECT_EQ(0, NumDiagnostics); -} - -TEST_F(VFSFromYAMLTest, ReturnsInternalPathVFSHit) { - IntrusiveRefCntPtr<vfs::InMemoryFileSystem> BaseFS( - new vfs::InMemoryFileSystem); - BaseFS->addFile("//root/foo/realname", 0, - MemoryBuffer::getMemBuffer("contents of a")); - auto FS = - getFromYAMLString("{ 'use-external-names': false,\n" - " 'roots': [\n" - "{\n" - " 'type': 'directory',\n" - " 'name': '//root/foo',\n" - " 'contents': [ {\n" - " 'type': 'file',\n" - " 'name': 'vfsname',\n" - " 'external-contents': 'realname'\n" - " }\n" - " ]\n" - "}]}", - BaseFS); - ASSERT_FALSE(FS->setCurrentWorkingDirectory("//root/foo")); - - auto OpenedF = FS->openFileForRead("vfsname"); - ASSERT_FALSE(OpenedF.getError()); - llvm::ErrorOr<std::string> Name = (*OpenedF)->getName(); - ASSERT_FALSE(Name.getError()); - EXPECT_EQ("vfsname", Name.get()); - - auto OpenedS = (*OpenedF)->status(); - ASSERT_FALSE(OpenedS.getError()); - EXPECT_EQ("vfsname", OpenedS->getName()); - EXPECT_TRUE(OpenedS->IsVFSMapped); - - auto DirectS = FS->status("vfsname"); - ASSERT_FALSE(DirectS.getError()); - EXPECT_EQ("vfsname", DirectS->getName()); - EXPECT_TRUE(DirectS->IsVFSMapped); - - EXPECT_EQ(0, NumDiagnostics); -} - TEST_F(VFSFromYAMLTest, CaseInsensitive) { IntrusiveRefCntPtr<DummyFileSystem> Lower(new DummyFileSystem()); Lower->addRegularFile("//root/foo/bar/a"); |