[llvm] Added clarifying comment to 'LLVMLinkInMCJIT' and 'LLVMLinkInInterpreter' (PR #92467)
Lang Hames via llvm-commits
llvm-commits at lists.llvm.org
Wed Jun 4 17:18:01 PDT 2025
lhames wrote:
Hi @Minding000. Sorry that I did not respond to your message earlier!
On reflection I think we should go with your original solution and just document the existing behavior: These functions are going to be deprecated and removed in the not too distant future anyway, at which point the problem goes away. In the mean time hiding them from the dylib interface would require clients to call them conditionally based on how they're linking LLVM, which seems like it'll be more error-prone (e.g. when people copy/paste example code they find that assumes dynamic linking).
If you're happy to restore your original comments I can go ahead and merge them.
Sorry for the run-around!
https://github.com/llvm/llvm-project/pull/92467
More information about the llvm-commits
mailing list