Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade to the latest LLVM #12

Open
2 tasks
langston-barrett opened this issue Aug 11, 2022 · 2 comments · Fixed by #124
Open
2 tasks

Upgrade to the latest LLVM #12

langston-barrett opened this issue Aug 11, 2022 · 2 comments · Fixed by #124

Comments

@langston-barrett
Copy link
Collaborator

cclyzer++ has only been tested with LLVM 10. Upgrading would involve:

  • Reading through the LLVM changelogs for any relevant changes in semantics
  • Upgrading the FactGenerator and C++ interfaces to the new LLVM API
@langston-barrett
Copy link
Collaborator Author

We should at least look into supporting multiple versions of LLVM. This makes cclyzer++ applicable in a wider variety of situations (e.g., a project where an older version of LLVM is required). More prosaically, it means we can keep the cclyzer++ submodule in the MATE project up-to-date without spending time upgrading MATE's LLVM support. This is valuable because MATE has a number of tests that could help find bugs in cclyzer++.

That being said, there might be major downsides to supporting multiple LLVM versions, the most likely candidate being an inordinate expense in terms of developer, build, or CI time; or in terms of (Docker) build complexity.

@langston-barrett
Copy link
Collaborator Author

Reopening because LLVM 15 isn't fully supported yet due to opaque pointers.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant