From c77e69d5007744bc5bcdf4df3e4c3faadf6cce83 Mon Sep 17 00:00:00 2001 From: David Blaikie Date: Fri, 27 Jun 2014 17:40:03 +0000 Subject: 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 --- tools/clang-format/ClangFormat.cpp | 2 +- tools/libclang/CIndex.cpp | 8 ++++---- tools/libclang/CIndexCodeCompletion.cpp | 4 ++-- tools/libclang/Indexing.cpp | 4 ++-- 4 files changed, 9 insertions(+), 9 deletions(-) (limited to 'tools') 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 FileNames(cl::Positional, cl::desc("[ ...]"), 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 == "-" ? "" : 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 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); } -- cgit v1.2.3