diff options
| author | Jonas Devlieghere <jonas@devlieghere.com> | 2019-02-14 18:05:43 +0000 |
|---|---|---|
| committer | Jonas Devlieghere <jonas@devlieghere.com> | 2019-02-14 18:05:43 +0000 |
| commit | ff04b3d4d72fdad7b3fd9f02e2be155ee5e2f058 (patch) | |
| tree | ef9b4f7b89bbf206b0cc447f2140b029745fa9fd /lldb/source/Plugins/ExpressionParser/Clang/ClangExpressionParser.h | |
| parent | b8b81e9b43c839f2761432e36927bb8d7b644b86 (diff) | |
| download | bcm5719-llvm-ff04b3d4d72fdad7b3fd9f02e2be155ee5e2f058.tar.gz bcm5719-llvm-ff04b3d4d72fdad7b3fd9f02e2be155ee5e2f058.zip | |
[ExpressionParser] Reuse the FileManager from the compiler instance.
I was looking at the ClangExpressionParser and noticed that we have a
FileManager owned by the expression parser and later ask the compiler
instance to create a new FileManager, owned by the clang CI. Looking at
the code I don't see a good reason for having two instances. This patch
removes the one owned by LLDB.
Differential revision: https://reviews.llvm.org/D58222
llvm-svn: 354041
Diffstat (limited to 'lldb/source/Plugins/ExpressionParser/Clang/ClangExpressionParser.h')
| -rw-r--r-- | lldb/source/Plugins/ExpressionParser/Clang/ClangExpressionParser.h | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/lldb/source/Plugins/ExpressionParser/Clang/ClangExpressionParser.h b/lldb/source/Plugins/ExpressionParser/Clang/ClangExpressionParser.h index 8dfff842646..e950a694306 100644 --- a/lldb/source/Plugins/ExpressionParser/Clang/ClangExpressionParser.h +++ b/lldb/source/Plugins/ExpressionParser/Clang/ClangExpressionParser.h @@ -178,8 +178,6 @@ private: std::unique_ptr<llvm::LLVMContext> m_llvm_context; ///< The LLVM context to generate IR into - std::unique_ptr<clang::FileManager> - m_file_manager; ///< The Clang file manager object used by the compiler std::unique_ptr<clang::CompilerInstance> m_compiler; ///< The Clang compiler used to parse expressions into IR std::unique_ptr<clang::CodeGenerator> |
