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

Lean: fix bug removing all fns before an extern fn #1019

Merged
merged 1 commit into from
Feb 16, 2025

Conversation

ineol
Copy link
Collaborator

@ineol ineol commented Feb 16, 2025

We were dropping all the translated functions as soon as we encountered an external function!

@ineol ineol added the Lean Issues with Sail to Lean translation label Feb 16, 2025
Copy link

github-actions bot commented Feb 16, 2025

Test Results

   13 files  ±0     27 suites  ±0   0s ⏱️ ±0s
  811 tests ±0    811 ✅ ±0  0 💤 ±0  0 ❌ ±0 
2 805 runs  ±0  2 805 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit e52132b. ± Comparison against base commit 8f90ea6.

♻️ This comment has been updated with latest results.

@Alasdair Alasdair merged commit a7fd409 into rems-project:sail2 Feb 16, 2025
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Lean Issues with Sail to Lean translation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants