Skip to content

Conversation

@treeowl
Copy link

@treeowl treeowl commented Apr 9, 2022

OK, against the right branch now.

oberblastmeister and others added 3 commits April 9, 2022 16:43
This one inlines the unboxed form into everything else, hopefully.
@oberblastmeister oberblastmeister merged commit 72510b4 into oberblastmeister:fast-intersection Apr 9, 2022
@treeowl
Copy link
Author

treeowl commented Apr 9, 2022

I would definitely suggest checking your benchmarks to make sure this isn't worse than what you were doing before, just in case there's something brittle. And if we can reduce the unboxed thing to INLINABLE without reducing perf that would certainly be nice.

@oberblastmeister
Copy link
Owner

The benchmarks stay the same

@treeowl
Copy link
Author

treeowl commented Apr 9, 2022

Awesome. You want to check what happens using intersectionWithKey# for the strict version?

@oberblastmeister
Copy link
Owner

Yeah I will

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