Skip to content

Commit

Permalink
set up interpreter after llvm
Browse files Browse the repository at this point in the history
This gives a little more time for the LLVM compile to run.  In practice,
it's so little it doesn't matter, but I think I'd rather have this in
than not.

Change-Id: Ifd813ac7024a2ead306c40471a9046dbd482baaa
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/275302
Reviewed-by: Mike Klein <mtklein@google.com>
Commit-Queue: Mike Klein <mtklein@google.com>
  • Loading branch information
Mike Klein authored and Skia Commit-Bot committed Mar 6, 2020
1 parent 491df6b commit 7405556
Showing 1 changed file with 5 additions and 1 deletion.
6 changes: 5 additions & 1 deletion src/core/SkVM.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -2198,12 +2198,16 @@ namespace skvm {
Program::Program(const std::vector<OptimizedInstruction>& interpreter,
const std::vector<OptimizedInstruction>& jit,
const std::vector<int>& strides,
const char* debug_name) : Program(interpreter, strides) {
const char* debug_name) : Program() {
fImpl->strides = strides;
#if 1 && defined(SKVM_LLVM)
this->setupLLVM(interpreter, debug_name);
#elif 1 && defined(SKVM_JIT)
this->setupJIT(jit, debug_name);
#endif

// Might as well do this after setupLLVM() to get a little more time to compile.
this->setupInterpreter(interpreter);
}

std::vector<InterpreterInstruction> Program::instructions() const { return fImpl->instructions; }
Expand Down

0 comments on commit 7405556

Please sign in to comment.