Skip to content

Conversation

@sjakobi
Copy link
Member

@sjakobi sjakobi commented Mar 12, 2022

This prevents unnecessary MArray boxing.

(Extracted from #230.)

This reverts commit 2ae034c.
@sjakobi
Copy link
Member Author

sjakobi commented Mar 12, 2022

Core and STG look a bit better. I can't make much sense of the C--, but at least it's a bit smaller and contains fewer heap pointer checks.

@sjakobi sjakobi marked this pull request as ready for review March 12, 2022 11:42
@sjakobi
Copy link
Member Author

sjakobi commented Mar 12, 2022

The All.HashMap.map benchmark was pretty useless, giving wildly varying timings. There's no indication that this patch makes them worse though.

@sjakobi sjakobi merged commit 5ea4197 into master Mar 14, 2022
@sjakobi sjakobi deleted the sjakobi/unbox-arrays-2-unit-loops branch March 14, 2022 17:03
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.

1 participant