This is needed for the corresponding MCJIT llvm-c patch I'm submitting to llvm-commits.
Created attachment 198980 [details]
I will wait with landing this until there is a clear story for my llvm-c patch.
Created attachment 198981 [details]
corresponding llvm patch
Comment on attachment 198980 [details]
It looks like LLVMFinalizeAllObjects() is not going to be a thing. Clearing r?.
It looks like this is again taking a different direction; this time, the LLVM C API will ensure that the MCJIT finalizes all objects before you use them. I'm leaving this bug open for now in case this again changes, but probably this will get resolved as wontfix.
Nope, it looks like LLVM isn't going to require this.