AuroraMiddleware/skia2
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

leak llvmcontexts for now

Browse Source 
Being thread_local, these destruct in an awkward order
with the other major thread_locals, the SkVMBlitter Program caches.

So, just leak them for now.

Change-Id: Ia45f65eba0a301e2a047458610d5fa3110dff006
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/274220
Reviewed-by: Mike Klein <mtklein@google.com>
Commit-Queue: Mike Klein <mtklein@google.com>
This commit is contained in:
Mike Klein 2020-02-28 15:27:13 -06:00 committed by Skia Commit-Bot
parent c02a41f0e6
commit c2b6769ab2
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/core/SkVM.cpp
View File

@ -1897,7 +1897,7 @@ namespace skvm {
#if defined(SKVM_LLVM)
void Program::setupLLVM(const std::vector<OptimizedInstruction>& instructions,
const char* debug_name) {
thread_local static llvm::LLVMContext ctx;
thread_local static llvm::LLVMContext& ctx = *(new llvm::LLVMContext);
auto mod = std::make_unique<llvm::Module>("", ctx);
// All the scary bare pointers from here on are owned by ctx or mod, I think.