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

Remove references to #parseKORE #3701

Merged
merged 1 commit into from
Dec 7, 2023
Merged

Remove references to #parseKORE #3701

merged 1 commit into from
Dec 7, 2023

Conversation

Scott-Guest
Copy link
Contributor

@Scott-Guest Scott-Guest commented Dec 6, 2023

This PR removes all references to the deprecated #parseKORE hook

Paired with runtimeverification/k#3850, runtimeverification/llvm-backend#919

@Scott-Guest Scott-Guest requested a review from Baltoli December 7, 2023 06:22
rv-jenkins pushed a commit to runtimeverification/k that referenced this pull request Dec 7, 2023
Per discussion in the frontend meeting, this PR removes `#parseKORE`
because it is

- deprecated in modern K in favor of Pyk-based approaches
- unused by all of our active semantics 
- type unsafe (and I am on a crusade to patch all type safety violations
:smile:)

Related: runtimeverification/llvm-backend#919,
runtimeverification/haskell-backend#3701
@Baltoli Baltoli marked this pull request as ready for review December 7, 2023 11:53
@rv-jenkins rv-jenkins merged commit a584730 into master Dec 7, 2023
7 checks passed
@rv-jenkins rv-jenkins deleted the parse-kore-no-more branch December 7, 2023 12:02
rv-jenkins pushed a commit to runtimeverification/llvm-backend that referenced this pull request Dec 7, 2023
This PR removes the deprecated `#parseKORE` hook.

Dependencies before merge: runtimeverification/k#3850,
runtimeverification/haskell-backend#3701
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