-
Notifications
You must be signed in to change notification settings - Fork 6.2k
8315066: Add unsigned bounds and known bits to TypeInt/Long #17508
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
Closed
Closed
Changes from all commits
Commits
Show all changes
83 commits
Select commit
Hold shift + click to select a range
3a87d66
implement unsigned bounds and known bits
merykitty 61f4d21
refactor
merykitty d11497e
fix template parameter
merykitty 12f268a
add unit tests
merykitty 6b417f9
fix tests, add verify
merykitty 756d615
add comments, group arguments to reduce C-style reference passing arg…
merykitty 1faa48b
fix release build
merykitty ffb0abd
Merge branch 'master' into unsignedbounds
merykitty 6e2e6c5
add comments
merykitty ec4ca3e
Merge branch 'master' into unsignedbounds
merykitty d5ad9f1
fix compile errors
merykitty 2c3807b
address reviews
merykitty ae47385
move static_asserts
merykitty 2bf545f
add more comments, group KnownBits
merykitty 4f4a6be
fix build
merykitty 8d14f8e
fix build
merykitty f164821
more explanation
merykitty 5990628
rename tests
merykitty 123e055
make should return the correct type
merykitty 089c566
add trivial test cases
merykitty 2e3955d
fix builds
merykitty e8ab32a
Merge branch 'master' into unsignedbounds
merykitty 9b70213
change (~v & ones) == 0 to (v & ones) == ones
merykitty 81f4e15
add doc to TypeInt, rename parameters, remove unused methods
merykitty a77e8f4
remove leftover code
merykitty 2564378
refine comments
merykitty 8a5370a
add comments, refactor functions to helper class
merykitty 644bced
address reviews
merykitty f2d3f3b
formality
merykitty c440a72
comment adjust_lo empty case
merykitty 4858e12
address reviews
merykitty 7f3316f
Merge branch 'master' into unsignedbounds
merykitty 41082f6
Merge branch 'master' into unsignedbounds
merykitty 468834f
further reviews
merykitty c2d7d36
whitespace
merykitty dcc9030
build failures
merykitty 7164653
build failure
merykitty 8c0ac2f
Merge branch 'master' into unsignedbounds
merykitty cf1de62
move try_cast to Type
merykitty 85acf6e
Merge branch 'master' into unsignedbounds
merykitty d033833
Merge branch 'master' into unsignedbounds
merykitty 4d33014
copyright
merykitty 539a032
Merge branch 'master' into unsignedbounds
merykitty c4d46c8
remove precompiled.hpp
merykitty 98aaa03
Merge branch 'master' into unsignedbounds
merykitty ac1ddfc
Emmanuel's review
merykitty f85eff5
make con
merykitty 01dc22d
exhaustive tests
kittyoracle c33576a
assignment operator
kittyoracle cf56091
refine first_violation
merykitty a849917
clean up intn_t
merykitty 1d34a54
include
merykitty ee07d29
number lemmas
merykitty d87e036
harden SimpleCanonicalResult
merykitty 0f347a5
Merge branch 'master' into unsignedbounds
merykitty 3cd2586
Merge branch 'master' into unsignedbounds
merykitty 5586036
refine comments
merykitty 7b26016
Merge branch 'master' into unsignedbounds
merykitty 727216f
reviews
merykitty 9ca8023
Merge branch 'master' into unsignedbounds
merykitty cdab191
Merge branch 'master' into unsignedbounds
merykitty 0fbbe5c
Explain what alignment means
merykitty fd7a7fa
More for Emanuel
merykitty eb3d69a
more rigour for ~hi
merykitty 7a72c6f
Merge branch 'master' into unsignedbounds
merykitty 547926e
formality for the non-existence case
merykitty 0eafb3a
alignment note
merykitty 1f53efa
grammar, more details for non-existence
merykitty 654f833
wording
merykitty e215d69
clarify where overflow comes from
merykitty 52d57e8
refer back to the formality section
merykitty b29ff4a
new_hi computation
merykitty 3a2aa8d
Emanuel's reviews
merykitty 5616c23
Emanuel's reviews
merykitty 58978fb
Emanuel's reviews
merykitty 95e5a23
intn_t refinements
merykitty 25a6f9b
add some more sanity static_asserts
merykitty 950a266
refine the cases where there does not exist a result
merykitty 693cec2
refinement
merykitty 56ffe4f
alignment wording
merykitty 6be30c5
Emanuel's reviews
merykitty 77aa406
add more intn_t tests
merykitty ad6ac7c
Merge branch 'master' into unsignedbounds
merykitty 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 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
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
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
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
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
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.
Uh oh!
There was an error while loading. Please reload this page.