Skip to content

Middlend Fixes#216

Open
DCupello1 wants to merge 9 commits intomainfrom
middlend-fixes
Open

Middlend Fixes#216
DCupello1 wants to merge 9 commits intomainfrom
middlend-fixes

Conversation

@DCupello1
Copy link
Copy Markdown

@DCupello1 DCupello1 commented Feb 10, 2026

This pull request is for some middlend fixes/changes that were done in the Rocq backend (#207) and are now being backported.

Fixes include:

  • Exposing wf relations, uncase removal projection functions, type families via hints in order to produce a better output in rocq.
  • In undep, made List IterE's with no iteration into a ListE in order for validation to work.
  • ImproveIds now actually tracks atoms in order to disambiguate them from variables.
  • New functionality in free (inter func)
  • New functionality in iter (now has visit arg func)
  • New functionality in walk (now has collect path func)
  • Small fix in sub which was missing a traversal in the expression of the coercion function.
  • After IL change, improveids did not have the correct transformation for mixops. This has now been fixed, along with changes to transforming ids with '#' which was added with the new fresh variables generation library.
  • Improveids: Added small fix for LetPrs to work.
  • Typefamilyremoval: Small fix to generated types to not clash with the base tf type.

@DCupello1
Copy link
Copy Markdown
Author

Rebased this to account for new IL changes. Still leaves improve ids not entirely fixed, will be done in a separate pull request.

@DCupello1
Copy link
Copy Markdown
Author

@nomeata could you look into this PR please? Since the IL refactor is done, these PRs can finally go in.

This one in particular is just really small changes so it shouldn't take long I hope.

@nomeata
Copy link
Copy Markdown

nomeata commented Feb 24, 2026

Hmm, I've paged out spectec stuff a bit these days, so I'm not eager and probably not able to give a good review. Maybe someone else whose currently playing with this could?

@DCupello1
Copy link
Copy Markdown
Author

Ah no worries then!

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