[PATCH] D83372: Fix for memory leak reported by Valgrind

[David Blaikie via Phabricator via llvm-commits]

dblaikie added a comment.

In D83372#2522877 <https://reviews.llvm.org/D83372#2522877>, @amaiorano wrote:

> Hello, this change would be very useful for SwiftShader <https://swiftshader.googlesource.com/SwiftShader>, a CPU-targeted Vulkan driver used by Chrome and Android (among other projects). This would plug memory leaks reported when unloading our driver. Can we get this reviewed/merged please?

What sort of situation are you dealing with/how does this issue manifest for you?

My understanding is that this would be a problem for users unloading/reloading a dynamic library (dll, so, dylib) but generally not an issue for static linking, for instance? & given, as @beanz said, llvm is unlikely to support re-use of llvm after shutdown, this doesn't seem like it'd go terribly well if you expect to be able to unload and then reload LLVM again later.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D83372/new/

https://reviews.llvm.org/D83372