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

Forbid [anywhere, symbolic] #3053

Merged
merged 2 commits into from
Nov 22, 2022
Merged

Forbid [anywhere, symbolic] #3053

merged 2 commits into from
Nov 22, 2022

Conversation

Baltoli
Copy link
Contributor

@Baltoli Baltoli commented Nov 21, 2022

The Haskell backend doesn't support anywhere rules, so we should forbid the attribute on symbolic rules as we already do for simplification rules: #2711

Doing so prevents the LLVM backend from crashing with an uninformative error message in certain contexts.

Fixes runtimeverification/llvm-backend#329

@rv-jenkins rv-jenkins changed the base branch from master to develop November 21, 2022 16:19
@dwightguth
Copy link
Collaborator

Seems fine but I'd like @ana-pantilie to confirm this won't break anything. I know in the past we were using anywhere for some stuff unrelated to anywhere rules and I just want to confirm that is no longer an issue.

@rv-jenkins rv-jenkins merged commit 5558dff into develop Nov 22, 2022
@rv-jenkins rv-jenkins deleted the anywhere-symbolic branch November 22, 2022 12:32
h0nzZik pushed a commit to h0nzZik/k that referenced this pull request Nov 24, 2022
…untimeverification#2590)

* haskell-backend/src/main/native/haskell-backend: d05a67040 - Re-export Control.Monad from Logic (runtimeverification#3053)

* haskell-backend/src/main/native/haskell-backend: 555776e31 - Commands history via Haskeline in the REPL (runtimeverification#3049)

* haskell-backend/src/main/native/haskell-backend: 544c24e96 - Add support for user-defined \\equals simplification (runtimeverification#3042)
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.

llvm-kompile crashes with bad error when #Ceil rule is added
4 participants