minimize inlining cost and code complexity #203
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
OLD: can inline popcntMaskSlice with cost 32 can inline popcntAndSlice with cost 32 can inline popcntOrSlice with cost 32 can inline popcntXorSlice with cost 32 github.com/bits-and-blooms/bitset.popcntMaskSlice STEXT size=165 args=0x30 locals=0x18 funcid=0x0 align=0x0 github.com/bits-and-blooms/bitset.popcntAndSlice STEXT size=158 args=0x30 locals=0x18 funcid=0x0 align=0x0 github.com/bits-and-blooms/bitset.popcntOrSlice STEXT size=158 args=0x30 locals=0x18 funcid=0x0 align=0x0 github.com/bits-and-blooms/bitset.popcntXorSlice STEXT size=158 args=0x30 locals=0x18 funcid=0x0 align=0x0 NEW: can inline popcntMaskSlice with cost 26 can inline popcntAndSlice with cost 26 can inline popcntOrSlice with cost 26 can inline popcntXorSlice with cost 26 github.com/bits-and-blooms/bitset.popcntMaskSlice STEXT nosplit size=82 args=0x30 locals=0x18 funcid=0x0 align=0x0 github.com/bits-and-blooms/bitset.popcntAndSlice STEXT nosplit size=79 args=0x30 locals=0x18 funcid=0x0 align=0x0 github.com/bits-and-blooms/bitset.popcntOrSlice STEXT nosplit size=79 args=0x30 locals=0x18 funcid=0x0 align=0x0 github.com/bits-and-blooms/bitset.popcntXorSlice STEXT nosplit size=79 args=0x30 locals=0x18 funcid=0x0 align=0x0
|
Merging. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
@lemire use the PR or close it as you like:
no explizit call of panic() for BCE, asm code is now
nosplit, no stack growth neededOLD:
NEW: