From 74055566bd14c9a5d374001ce7530f7b38e7d55d Mon Sep 17 00:00:00 2001 From: Mike Klein Date: Thu, 5 Mar 2020 11:13:23 -0600 Subject: [PATCH] set up interpreter after llvm 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 Commit-Queue: Mike Klein --- src/core/SkVM.cpp | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/src/core/SkVM.cpp b/src/core/SkVM.cpp index 67edfe6b91393..b6ed2601fc53e 100644 --- a/src/core/SkVM.cpp +++ b/src/core/SkVM.cpp @@ -2198,12 +2198,16 @@ namespace skvm { Program::Program(const std::vector& interpreter, const std::vector& jit, const std::vector& 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 Program::instructions() const { return fImpl->instructions; }