BinaryAnalysisPlatform / bap

Binary Analysis Platform
MIT License
2.07k stars 273 forks source link

Fix compatibility with LLVM 16 #1595

Closed katrinafyi closed 9 months ago

katrinafyi commented 1 year ago

LLVM 16 uses std::optional in place of llvm::Optional. Since these have very similar interfaces, we use auto to maintain backwards compatibility.

I haven't tested much, but it compiles successfully with LLVM 16 and LLVM 15.

ivg commented 9 months ago

covered by #1597