tree 415884506f68987d25fad7b64bc317d00b078d59
parent 4bc9f6e59266a28ef37f2cea20b31dc967195060
author Ben Clayton <bclayton@google.com> 1603301853 +0100
committer Ben Clayton <bclayton@google.com> 1603310117 +0000

LLVMJIT: Fix memory leak

We were calling `new MemoryMapper` and passing it into the `llvm::SectionMemoryManager`, but the `SectionMemoryManager` does not take ownership, resulting in a leak.

Given that the `MemoryMapper` is entirely stateless, just use a pointer to a single static instance.

Bug: b/171402030
Change-Id: Ic3a118a5f4aeea6b6b60861399993e81827c9d48
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/49488
Reviewed-by: Antonio Maiorano <amaiorano@google.com>
Reviewed-by: Nicolas Capens <nicolascapens@google.com>
Kokoro-Result: kokoro <noreply+kokoro@google.com>
Tested-by: Ben Clayton <bclayton@google.com>