Skip to content

Parse spec directly #1580

Merged
pavpanchekha merged 3 commits intomainfrom
spec-split
May 7, 2026
Merged

Parse spec directly #1580
pavpanchekha merged 3 commits intomainfrom
spec-split

Conversation

@JonasRegehr
Copy link
Copy Markdown
Collaborator

@JonasRegehr JonasRegehr commented Apr 14, 2026

  • Adds fpcore->spec, which parses fpcore directly into a spec
  • Changes some usages of fpcore->prog to fpcore->spec

@JonasRegehr JonasRegehr marked this pull request as ready for review April 17, 2026 20:42
Comment thread src/syntax/sugar.rkt Outdated
Comment thread src/syntax/sugar.rkt Outdated
@pavpanchekha
Copy link
Copy Markdown
Contributor

I'm a little bit worried about fpcore->spec is missing something but otherwise this is obviously a good PR.

@pavpanchekha
Copy link
Copy Markdown
Contributor

This is going to end up having a bit of an overlap with #1484. Happy to merge either one first but both are important & valuable PRs.

@pavpanchekha
Copy link
Copy Markdown
Contributor

Ok, thinking this over, I think it works except helper functions / named FPCores. Ideally we'd have those expand into their underlying spec; doing that will require referencing the platform.

# Conflicts:
#	src/api/run.rkt
#	src/syntax/platform.rkt
#	src/syntax/read.rkt
@pavpanchekha pavpanchekha merged commit 0ffc4a4 into main May 7, 2026
6 checks passed
@pavpanchekha pavpanchekha deleted the spec-split branch May 7, 2026 14:58
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