Skip to content

Handle more recent versions of LLVM in the build scripts#1751

Merged
MartinNowack merged 3 commits intoklee:masterfrom
ccadar:build
Nov 5, 2024
Merged

Handle more recent versions of LLVM in the build scripts#1751
MartinNowack merged 3 commits intoklee:masterfrom
ccadar:build

Conversation

@ccadar
Copy link
Contributor

@ccadar ccadar commented Oct 31, 2024

Summary:

Handle more recent versions of LLVM in the build scripts. This has enabled building Docker images for LLVM 17-19.

Checklist:

  • The PR addresses a single issue. If it can be divided into multiple independent PRs, please do so.
  • The PR is divided into a logical sequence of commits OR a single commit is sufficient.
  • There are no unnecessary commits (e.g. commits fixing issues in a previous commit in the same PR).
  • Each commit has a meaningful message documenting what it does.
  • All messages added to the codebase, all comments, as well as commit messages are spellchecked.
  • The code is commented OR not applicable/necessary.
  • The patch is formatted via clang-format OR not applicable (if explicitly overridden leave unchecked and explain).
  • There are test cases for the code you added or modified OR no such test cases are required.

@ccadar ccadar force-pushed the build branch 2 times, most recently from 2580228 to a56d92e Compare October 31, 2024 17:38
@ccadar ccadar force-pushed the build branch 2 times, most recently from 53021b2 to f42e49a Compare November 5, 2024 11:27
@ccadar ccadar changed the title Handle more recent versions of LLVM in the build scripts and add them to the CI Handle more recent versions of LLVM in the build scripts Nov 5, 2024
Copy link
Contributor

@MartinNowack MartinNowack left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ccadar Thanks for the effort and building the new images.
Looks good - going to merge.

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 this pull request may close these issues.

2 participants