[llvm] 67ffce6 - Make WindowsManifestMerger::merge() take a MemoryBufferRef
Nico Weber via llvm-commits
llvm-commits at lists.llvm.org
Tue Aug 24 13:39:31 PDT 2021
Author: Nico Weber
Date: 2021-08-24T16:39:20-04:00
New Revision: 67ffce68bc3e95be4cfad0493a2f55a3984e4851
URL: https://github.com/llvm/llvm-project/commit/67ffce68bc3e95be4cfad0493a2f55a3984e4851
DIFF: https://github.com/llvm/llvm-project/commit/67ffce68bc3e95be4cfad0493a2f55a3984e4851.diff
LOG: Make WindowsManifestMerger::merge() take a MemoryBufferRef
No behavior change.
Added:
Modified:
llvm/include/llvm/WindowsManifest/WindowsManifestMerger.h
llvm/lib/WindowsManifest/WindowsManifestMerger.cpp
llvm/tools/llvm-mt/llvm-mt.cpp
Removed:
################################################################################
diff --git a/llvm/include/llvm/WindowsManifest/WindowsManifestMerger.h b/llvm/include/llvm/WindowsManifest/WindowsManifestMerger.h
index 31f4daeb7019..2da74bb9dce8 100644
--- a/llvm/include/llvm/WindowsManifest/WindowsManifestMerger.h
+++ b/llvm/include/llvm/WindowsManifest/WindowsManifestMerger.h
@@ -30,6 +30,7 @@
namespace llvm {
class MemoryBuffer;
+class MemoryBufferRef;
namespace windows_manifest {
@@ -49,7 +50,7 @@ class WindowsManifestMerger {
public:
WindowsManifestMerger();
~WindowsManifestMerger();
- Error merge(const MemoryBuffer &Manifest);
+ Error merge(MemoryBufferRef Manifest);
// Returns vector containing merged xml manifest, or uninitialized vector for
// empty manifest.
diff --git a/llvm/lib/WindowsManifest/WindowsManifestMerger.cpp b/llvm/lib/WindowsManifest/WindowsManifestMerger.cpp
index 6af7bc699d05..1be1d34417eb 100644
--- a/llvm/lib/WindowsManifest/WindowsManifestMerger.cpp
+++ b/llvm/lib/WindowsManifest/WindowsManifestMerger.cpp
@@ -35,7 +35,7 @@ void WindowsManifestError::log(raw_ostream &OS) const { OS << Msg; }
class WindowsManifestMerger::WindowsManifestMergerImpl {
public:
~WindowsManifestMergerImpl();
- Error merge(const MemoryBuffer &Manifest);
+ Error merge(MemoryBufferRef Manifest);
std::unique_ptr<MemoryBuffer> getMergedManifest();
private:
@@ -620,7 +620,7 @@ WindowsManifestMerger::WindowsManifestMergerImpl::~WindowsManifestMergerImpl() {
}
Error WindowsManifestMerger::WindowsManifestMergerImpl::merge(
- const MemoryBuffer &Manifest) {
+ MemoryBufferRef Manifest) {
if (Merged)
return make_error<WindowsManifestError>(
"merge after getMergedManifest is not supported");
@@ -690,7 +690,7 @@ WindowsManifestMerger::WindowsManifestMergerImpl::~WindowsManifestMergerImpl() {
}
Error WindowsManifestMerger::WindowsManifestMergerImpl::merge(
- const MemoryBuffer &Manifest) {
+ MemoryBufferRef Manifest) {
return make_error<WindowsManifestError>("no libxml2");
}
@@ -708,7 +708,7 @@ WindowsManifestMerger::WindowsManifestMerger()
WindowsManifestMerger::~WindowsManifestMerger() {}
-Error WindowsManifestMerger::merge(const MemoryBuffer &Manifest) {
+Error WindowsManifestMerger::merge(MemoryBufferRef Manifest) {
return Impl->merge(Manifest);
}
diff --git a/llvm/tools/llvm-mt/llvm-mt.cpp b/llvm/tools/llvm-mt/llvm-mt.cpp
index 1ff66ee8c1c5..df1ecacf9e05 100644
--- a/llvm/tools/llvm-mt/llvm-mt.cpp
+++ b/llvm/tools/llvm-mt/llvm-mt.cpp
@@ -135,8 +135,7 @@ int main(int Argc, const char **Argv) {
MemoryBuffer::getFile(File);
if (!ManifestOrErr)
reportError(File, ManifestOrErr.getError());
- MemoryBuffer &Manifest = *ManifestOrErr.get();
- error(Merger.merge(Manifest));
+ error(Merger.merge(*ManifestOrErr.get()));
}
std::unique_ptr<MemoryBuffer> OutputBuffer = Merger.getMergedManifest();
More information about the llvm-commits
mailing list