-
Notifications
You must be signed in to change notification settings - Fork 187
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
Constant evaluation #649
Constant evaluation #649
Conversation
Codecov Report
@@ Coverage Diff @@
## master #649 +/- ##
==========================================
- Coverage 86.80% 85.81% -1.00%
==========================================
Files 96 97 +1
Lines 8627 9075 +448
==========================================
+ Hits 7489 7788 +299
- Misses 1138 1287 +149
Continue to review full report at Codecov.
|
5d3a9bc
to
9472cc5
Compare
ab18349
to
2c52e27
Compare
bf0184b
to
769fe02
Compare
64c7dfd
to
2ae6870
Compare
3f927f6
to
c307469
Compare
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 work!
c307469
to
ab61912
Compare
Fixed an invalid example in news fragments. |
let my_array: Array<i32, { GLOBAL if USE_GLOBAL else 4 }> | ||
``` | ||
|
||
# Bug fixes |
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.
Just a nitpick but next time it would be better to create another file 649.bugfix.md
because our changelog generator picks up these file name keywords to group the items accordingly. No big deal though, we can always apply changes to the changelog by hand.
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.
Oh, Ok. I understand.
This PR adds some features and fixes some ICE related to constant evaluation.
Features
1. Support local constant
Example:
2. Support constant expression
Example:
3. Support constant generics expression
Example:
Bug fixes
Fix ICE when constant type is a mismatch
Example:
Fix ICE when assigning value to constant twice
Example:
TODO
I opened a follow-up issue here