diff options
author | David Blaikie <dblaikie@gmail.com> | 2014-06-27 17:40:03 +0000 |
---|---|---|
committer | David Blaikie <dblaikie@gmail.com> | 2014-06-27 17:40:03 +0000 |
commit | c77e69d5007744bc5bcdf4df3e4c3faadf6cce83 (patch) | |
tree | 86e07297f8e9bf165060657975666fe2e64bb999 /tools | |
parent | 54d7831c3b8f99ed297abd939f1cd6a884c03a99 (diff) | |
download | clang-c77e69d5007744bc5bcdf4df3e4c3faadf6cce83.tar.gz clang-c77e69d5007744bc5bcdf4df3e4c3faadf6cce83.tar.bz2 clang-c77e69d5007744bc5bcdf4df3e4c3faadf6cce83.tar.xz |
Remove 'const' from MemoryBuffers used through the SourceManager
This removes a const_cast added in r211884 that occurred due to an
inconsistency in how MemoryBuffers are handled between some parts of
clang and LLVM.
MemoryBuffers are immutable and the general convention in the LLVM
project is to omit const from immutable types as it's simply
redundant/verbose (see llvm::Type, for example). While this change
doesn't remove "const" from /every/ MemoryBuffer, it at least makes this
chain of ownership/usage consistent.
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@211915 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'tools')
-rw-r--r-- | tools/clang-format/ClangFormat.cpp | 2 | ||||
-rw-r--r-- | tools/libclang/CIndex.cpp | 8 | ||||
-rw-r--r-- | tools/libclang/CIndexCodeCompletion.cpp | 4 | ||||
-rw-r--r-- | tools/libclang/Indexing.cpp | 4 |
4 files changed, 9 insertions, 9 deletions
diff --git a/tools/clang-format/ClangFormat.cpp b/tools/clang-format/ClangFormat.cpp index e0239499af..575ac7a93d 100644 --- a/tools/clang-format/ClangFormat.cpp +++ b/tools/clang-format/ClangFormat.cpp @@ -103,7 +103,7 @@ static cl::list<std::string> FileNames(cl::Positional, cl::desc("[<file> ...]"), namespace clang { namespace format { -static FileID createInMemoryFile(StringRef FileName, const MemoryBuffer *Source, +static FileID createInMemoryFile(StringRef FileName, MemoryBuffer *Source, SourceManager &Sources, FileManager &Files) { const FileEntry *Entry = Files.getVirtualFile(FileName == "-" ? "<stdin>" : FileName, diff --git a/tools/libclang/CIndex.cpp b/tools/libclang/CIndex.cpp index fdd6334bac..8683ae62b0 100644 --- a/tools/libclang/CIndex.cpp +++ b/tools/libclang/CIndex.cpp @@ -2787,8 +2787,8 @@ static void clang_parseTranslationUnit_Impl(void *UserData) { for (unsigned I = 0; I != num_unsaved_files; ++I) { StringRef Data(unsaved_files[I].Contents, unsaved_files[I].Length); - const llvm::MemoryBuffer *Buffer - = llvm::MemoryBuffer::getMemBufferCopy(Data, unsaved_files[I].Filename); + llvm::MemoryBuffer *Buffer = + llvm::MemoryBuffer::getMemBufferCopy(Data, unsaved_files[I].Filename); RemappedFiles->push_back(std::make_pair(unsaved_files[I].Filename, Buffer)); } @@ -3073,8 +3073,8 @@ static void clang_reparseTranslationUnit_Impl(void *UserData) { for (unsigned I = 0; I != num_unsaved_files; ++I) { StringRef Data(unsaved_files[I].Contents, unsaved_files[I].Length); - const llvm::MemoryBuffer *Buffer - = llvm::MemoryBuffer::getMemBufferCopy(Data, unsaved_files[I].Filename); + llvm::MemoryBuffer *Buffer = + llvm::MemoryBuffer::getMemBufferCopy(Data, unsaved_files[I].Filename); RemappedFiles->push_back(std::make_pair(unsaved_files[I].Filename, Buffer)); } diff --git a/tools/libclang/CIndexCodeCompletion.cpp b/tools/libclang/CIndexCodeCompletion.cpp index edc8368ca3..9dd2dfec7a 100644 --- a/tools/libclang/CIndexCodeCompletion.cpp +++ b/tools/libclang/CIndexCodeCompletion.cpp @@ -695,8 +695,8 @@ void clang_codeCompleteAt_Impl(void *UserData) { SmallVector<ASTUnit::RemappedFile, 4> RemappedFiles; for (unsigned I = 0; I != num_unsaved_files; ++I) { StringRef Data(unsaved_files[I].Contents, unsaved_files[I].Length); - const llvm::MemoryBuffer *Buffer - = llvm::MemoryBuffer::getMemBufferCopy(Data, unsaved_files[I].Filename); + llvm::MemoryBuffer *Buffer = + llvm::MemoryBuffer::getMemBufferCopy(Data, unsaved_files[I].Filename); RemappedFiles.push_back(std::make_pair(unsaved_files[I].Filename, Buffer)); } diff --git a/tools/libclang/Indexing.cpp b/tools/libclang/Indexing.cpp index 6bb995ae5a..24ab7b206c 100644 --- a/tools/libclang/Indexing.cpp +++ b/tools/libclang/Indexing.cpp @@ -592,8 +592,8 @@ static void clang_indexSourceFile_Impl(void *UserData) { for (unsigned I = 0; I != num_unsaved_files; ++I) { StringRef Data(unsaved_files[I].Contents, unsaved_files[I].Length); - const llvm::MemoryBuffer *Buffer - = llvm::MemoryBuffer::getMemBufferCopy(Data, unsaved_files[I].Filename); + llvm::MemoryBuffer *Buffer = + llvm::MemoryBuffer::getMemBufferCopy(Data, unsaved_files[I].Filename); CInvok->getPreprocessorOpts().addRemappedFile(unsaved_files[I].Filename, Buffer); BufOwner->Buffers.push_back(Buffer); } |