> On 21 Feb 2023, at 15:50, Heikki Linnakangas <hlinn...@iki.fi> wrote: > > llvm_release_context() calls llvm_enter_fatal_on_oom(), but it never calls > llvm_leave_fatal_on_oom(). Isn't that a clear leak?
Not sure how much of a leak it is since IIUC LLVM just stores a function pointer to our error handler, but I can't see a reason not clean it up here. The attached fix LGTM and passes make check with jit_above_cost set to zero. -- Daniel Gustafsson
