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

Update dependency: llvm-backend/src/main/native/llvm-backend #3883

Merged
merged 10 commits into from
Dec 20, 2023

Conversation

rv-jenkins
Copy link
Contributor

No description provided.

rv-jenkins pushed a commit to runtimeverification/llvm-backend that referenced this pull request Dec 20, 2023
The frontend test suite catches a bug introduced in #933, where byte
literal values that overflow a signed character are printed incorrectly.
I took a look at the best place to test this (i.e. whether it was worth
backporting the relevant test into the backend test suite), and it is
indeed in the frontend because of the way that we provide test cases in
each project.

I have run the downstream K test suite and verified that it passes with
this fix applied.

Fixes: runtimeverification/k#3883
@Baltoli Baltoli reopened this Dec 20, 2023
@Baltoli
Copy link
Contributor

Baltoli commented Dec 20, 2023

CI got confused here when I accidentally closed the PR branch; I had to push the commit manually to get CI to get the update and trigger a run

@rv-jenkins rv-jenkins merged commit 81d0f42 into develop Dec 20, 2023
9 checks passed
@rv-jenkins rv-jenkins deleted the _update-deps/runtimeverification/llvm-backend branch December 20, 2023 13:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants