Support linking with LLVM shared libraries

BinaryAnalysisPlatform / bap

Binary Analysis Platform

MIT License

2.07k stars 273 forks source link

Support linking with LLVM shared libraries #1593

Closed drvink closed 9 months ago

drvink commented 1 year ago

This must be manually enabled by setting the variable llvm-shared to a true value in the current opam switch, similar to the existing llvm-config variable used by conf-bap-llvm.

bap_llvm/config/llvm_configurator.ml: use llvm-config --shared-mode to get the appropriate linking mode
bap_llvm/llvm_disasm.cpp: add missing include for Optional.h

drvink commented 9 months ago

Is anything preventing this from being merged?

ivg commented 9 months ago

Is anything preventing this from being merged?

No, not really. There was an issue with the build action, but it looks like it wasn't the problem of this PR, I will look at it later! Thanks for the contribution!