Skip to content

transpile: Include enums in CastKind::from_types#1737

Open
Rua wants to merge 2 commits intoimmunant:masterfrom
Rua:enum-castkind
Open

transpile: Include enums in CastKind::from_types#1737
Rua wants to merge 2 commits intoimmunant:masterfrom
Rua:enum-castkind

Conversation

@Rua
Copy link
Copy Markdown
Contributor

@Rua Rua commented Apr 15, 2026

This wasn't catching enum types yet, so they'd end up defaulting to BitCast.

Copy link
Copy Markdown
Contributor

@fw-immunant fw-immunant left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The code changes look plausible, but the practical impact of this change isn't clear to me. Does this impact generated code or enable accepting more input programs? It would be nice to have a test demonstrating changed behavior.

@Rua
Copy link
Copy Markdown
Contributor Author

Rua commented Apr 15, 2026

It's a bugfix, and it's needed to make #1620 work, since #1703 merged.

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