-
Notifications
You must be signed in to change notification settings - Fork 483
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Kwxm/costing/bitwise 4 #6301
Merged
Merged
Kwxm/costing/bitwise 4 #6301
Changes from 23 commits
Commits
Show all changes
24 commits
Select commit
Hold shift + click to select a range
4db0888
Costing for new bitwise builtins
kwxm a2b3868
Fix alignment problem
kwxm e4fa023
Add changelog entry
kwxm 9386dfd
Update some CIP URLs
kwxm 1c763cd
Update some CIP URLs
kwxm 1ea93b0
Remove outdated comment
kwxm d927468
Address some review comments
kwxm e231192
Address some review comments
kwxm 34be0a8
Merge branch 'master' into kwxm/costing/bitwise-4
kwxm 86cf73b
Fix for changes in main branch
kwxm b1006cf
Merge branch 'master' into kwxm/costing/bitwise-4
kwxm 794bc8c
Merge branch 'master' into kwxm/costing/bitwise-4
kwxm e99a354
Update costing for revised version of writeBits
kwxm 043eb8b
Fix benchmark tests for integer division builtins
kwxm 8c94930
Fix comments
kwxm fcb267f
Update benchmark results for writeBits split updates
kwxm e83cb90
Try removing zip in writeBits
kwxm 76d6c0c
Remove wrapper
kwxm abc05f3
Rearrange code again
kwxm e4015ab
Tidying up
kwxm ac9a27c
Fix plutus-tx
kwxm 72b4228
Make some helpers INLINEABLE
kwxm 63b16f2
Rearrange code again
kwxm bb345fc
Simplify known type instances for 'ListCostedByLength'
effectfully File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains 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
This file contains 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
This file contains 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
This file contains 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
3 changes: 3 additions & 0 deletions
3
plutus-core/changelog.d/20240711_030856_kenneth.mackenzie_bitwise_4.md
This file contains 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,3 @@ | ||
### Added | ||
|
||
- Added costing for the new bitwise builtins (see CIP-0058), which will probably become available at the Chang+1 HF. |
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great rename by the way.