Fixes for LLVM 17 (and 16)

BinaryAnalysisPlatform / bap

Binary Analysis Platform

MIT License

2.07k stars 273 forks source link

Fixes for LLVM 17 (and 16) #1597

Closed bmourad01 closed 8 months ago

bmourad01 commented 8 months ago

This also covers #1595.

There was a really tricky segfault to track down in the call to MCInstrDesc::mayAffectControlFlow, see my comment as well as here. This comment is also present in LLVM 16.

ivg commented 8 months ago

Thanks! The build on linux is failing with No space left on device on z3. I will take a look at it tomorrow to see if we can squeeze anything out of the runner. However, the build on macOS fails with llvm, it looks like an issue with the packager, will try addressing it in a separate PR.