Commit c9ea4ca1 authored by Robert Kimball's avatar Robert Kimball

fix per review comments

parent a23b9241
...@@ -74,9 +74,9 @@ Compiler::~Compiler() ...@@ -74,9 +74,9 @@ Compiler::~Compiler()
{ {
} }
void Compiler::set_precomiled_header_source(const std::string& source) void Compiler::set_precompiled_header_source(const std::string& source)
{ {
s_static_compiler.set_precomiled_header_source(source); s_static_compiler.set_precompiled_header_source(source);
} }
std::unique_ptr<llvm::Module> Compiler::compile(const std::string& source) std::unique_ptr<llvm::Module> Compiler::compile(const std::string& source)
...@@ -316,7 +316,7 @@ std::unique_ptr<llvm::Module> StaticCompiler::compile(const string& source) ...@@ -316,7 +316,7 @@ std::unique_ptr<llvm::Module> StaticCompiler::compile(const string& source)
// Preprocessor options // Preprocessor options
auto& PPO = m_compiler->getInvocation().getPreprocessorOpts(); auto& PPO = m_compiler->getInvocation().getPreprocessorOpts();
PPO.ImplicitPCHInclude = m_pch_path; PPO.ImplicitPCHInclude = m_pch_path;
PPO.DisablePCHValidation = 1; PPO.DisablePCHValidation = 0;
} }
// Map code filename to a memoryBuffer // Map code filename to a memoryBuffer
......
...@@ -51,7 +51,7 @@ class ngraph::codegen::Compiler ...@@ -51,7 +51,7 @@ class ngraph::codegen::Compiler
public: public:
Compiler(); Compiler();
~Compiler(); ~Compiler();
void set_precomiled_header_source(const std::string& source); void set_precompiled_header_source(const std::string& source);
std::unique_ptr<llvm::Module> compile(const std::string& source); std::unique_ptr<llvm::Module> compile(const std::string& source);
private: private:
...@@ -65,7 +65,7 @@ public: ...@@ -65,7 +65,7 @@ public:
void set_debuginfo_enabled(bool state) { m_debuginfo_enabled = state; } void set_debuginfo_enabled(bool state) { m_debuginfo_enabled = state; }
bool is_debuginfo_enabled() { return m_debuginfo_enabled; } bool is_debuginfo_enabled() { return m_debuginfo_enabled; }
void set_precomiled_header_source(const std::string& source) void set_precompiled_header_source(const std::string& source)
{ {
m_precomiled_header_source = source; m_precomiled_header_source = source;
} }
......
...@@ -319,7 +319,7 @@ using namespace ngraph::runtime::cpu::eigen; ...@@ -319,7 +319,7 @@ using namespace ngraph::runtime::cpu::eigen;
codegen::Compiler compiler; codegen::Compiler compiler;
codegen::ExecutionEngine execution_engine; codegen::ExecutionEngine execution_engine;
compiler.set_precomiled_header_source(pch_header_source); compiler.set_precompiled_header_source(pch_header_source);
auto llvm_module = compiler.compile(code); auto llvm_module = compiler.compile(code);
if (llvm_module == nullptr) if (llvm_module == nullptr)
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment