Skip to content

Conversation

soulomoon
Copy link
Collaborator

@soulomoon soulomoon commented Aug 21, 2025

Fix #4699

Try to use the fat interface type for core from ghc.

@soulomoon soulomoon linked an issue Aug 21, 2025 that may be closed by this pull request
@soulomoon soulomoon marked this pull request as ready for review August 22, 2025 00:51
@soulomoon soulomoon requested a review from wz1000 as a code owner August 22, 2025 00:51
@wz1000
Copy link
Collaborator

wz1000 commented Aug 22, 2025

Do we still need the custom logic for any older ghc version?

Otherwise this patch looks good, thanks for doing this!

Copy link
Collaborator

@fendor fendor left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for catching this!

@fendor
Copy link
Collaborator

fendor commented Aug 22, 2025

@wz1000 CI is green, so apparently no?

@wz1000
Copy link
Collaborator

wz1000 commented Aug 22, 2025

TopIfaceBinding can also be removed?

@soulomoon
Copy link
Collaborator Author

Indeed, TopIfaceBinding removed.

@fendor fendor enabled auto-merge (squash) August 22, 2025 10:11
@fendor fendor merged commit 0c73c2b into master Aug 22, 2025
59 of 64 checks passed
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.

Redundant toIfaceIdDetails log
3 participants