]> granicus.if.org Git - llvm/commitdiff
Make SourceManager::createFileID(UnownedTag, ...) take a const llvm::MemoryBuffer*
authorNico Weber <nicolasweber@gmx.de>
Thu, 4 Apr 2019 21:06:41 +0000 (21:06 +0000)
committerNico Weber <nicolasweber@gmx.de>
Thu, 4 Apr 2019 21:06:41 +0000 (21:06 +0000)
Requires making the llvm::MemoryBuffer* stored by SourceManager const,
which in turn requires making the accessors for that return const
llvm::MemoryBuffer*s and updating all call sites.

The original motivation for this was to use it and fix the TODO in
CodeGenAction.cpp's ConvertBackendLocation() by using the UnownedTag
version of createFileID, and since llvm::SourceMgr* hands out a const
llvm::MemoryBuffer* this is required. I'm not sure if fixing the TODO
this way actually works, but this seems like a good change on its own
anyways.

No intended behavior change.

Differential Revision: https://reviews.llvm.org/D60247

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@357724 91177308-0d34-0410-b5e6-96231b3b80d8

include/llvm/Support/MemoryBuffer.h

index 44052697d06c662b08b87c6a306fb509268ad907..fc327d2fbd7a24116b8c55464831d54eda25c796 100644 (file)
@@ -264,7 +264,7 @@ class MemoryBufferRef {
 
 public:
   MemoryBufferRef() = default;
-  MemoryBufferRef(MemoryBuffer& Buffer)
+  MemoryBufferRef(const MemoryBuffer& Buffer)
       : Buffer(Buffer.getBuffer()), Identifier(Buffer.getBufferIdentifier()) {}
   MemoryBufferRef(StringRef Buffer, StringRef Identifier)
       : Buffer(Buffer), Identifier(Identifier) {}