diff options
author | Alp Toker <alp@nuanti.com> | 2014-05-21 01:12:41 +0000 |
---|---|---|
committer | Alp Toker <alp@nuanti.com> | 2014-05-21 01:12:41 +0000 |
commit | b671e34c36463f972a3984e407f91484ad6e5806 (patch) | |
tree | cc9796f60ac6ea4e73fc15bc58642210a590e7b6 /clang/unittests/Basic/SourceManagerTest.cpp | |
parent | 80fd10e6e10d23fed1a78cc3755ce06bcc2cb9d8 (diff) | |
download | llvm-b671e34c36463f972a3984e407f91484ad6e5806.zip llvm-b671e34c36463f972a3984e407f91484ad6e5806.tar.gz llvm-b671e34c36463f972a3984e407f91484ad6e5806.tar.bz2 |
SourceManager: Use setMainFileID() consistently
Eliminate createMainFileID() / createMainFileIDForMemBuffer() utility
functions. These didn't add much convenience and conflated two distinct
operations.
This change makes things easier to follow by providing a consistent interface
and getting rid of a bunch of cast-to-voids.
llvm-svn: 209266
Diffstat (limited to 'clang/unittests/Basic/SourceManagerTest.cpp')
-rw-r--r-- | clang/unittests/Basic/SourceManagerTest.cpp | 11 |
1 files changed, 7 insertions, 4 deletions
diff --git a/clang/unittests/Basic/SourceManagerTest.cpp b/clang/unittests/Basic/SourceManagerTest.cpp index 36e6a557..f227d0d 100644 --- a/clang/unittests/Basic/SourceManagerTest.cpp +++ b/clang/unittests/Basic/SourceManagerTest.cpp @@ -75,7 +75,8 @@ TEST_F(SourceManagerTest, isBeforeInTranslationUnit) { "#define M(x) [x]\n" "M(foo)"; MemoryBuffer *buf = MemoryBuffer::getMemBuffer(source); - FileID mainFileID = SourceMgr.createMainFileIDForMemBuffer(buf); + FileID mainFileID = SourceMgr.createFileID(buf); + SourceMgr.setMainFileID(mainFileID); VoidModuleLoader ModLoader; HeaderSearch HeaderInfo(new HeaderSearchOptions, SourceMgr, Diags, LangOpts, @@ -127,7 +128,8 @@ TEST_F(SourceManagerTest, getColumnNumber) { "int y;"; MemoryBuffer *Buf = MemoryBuffer::getMemBuffer(Source); - FileID MainFileID = SourceMgr.createMainFileIDForMemBuffer(Buf); + FileID MainFileID = SourceMgr.createFileID(Buf); + SourceMgr.setMainFileID(MainFileID); bool Invalid; @@ -186,7 +188,8 @@ TEST_F(SourceManagerTest, getMacroArgExpandedLocation) { MemoryBuffer *headerBuf = MemoryBuffer::getMemBuffer(header); MemoryBuffer *mainBuf = MemoryBuffer::getMemBuffer(main); - FileID mainFileID = SourceMgr.createMainFileIDForMemBuffer(mainBuf); + FileID mainFileID = SourceMgr.createFileID(mainBuf); + SourceMgr.setMainFileID(mainFileID); const FileEntry *headerFile = FileMgr.getVirtualFile("/test-header.h", headerBuf->getBufferSize(), 0); @@ -284,7 +287,7 @@ TEST_F(SourceManagerTest, isBeforeInTranslationUnitWithMacroInInclude) { MemoryBuffer *headerBuf = MemoryBuffer::getMemBuffer(header); MemoryBuffer *mainBuf = MemoryBuffer::getMemBuffer(main); - SourceMgr.createMainFileIDForMemBuffer(mainBuf); + SourceMgr.setMainFileID(SourceMgr.createFileID(mainBuf)); const FileEntry *headerFile = FileMgr.getVirtualFile("/test-header.h", headerBuf->getBufferSize(), 0); |