Skip to content
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.

Sign up for GitHub

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

Jump to bottom

bugfix: issue #4054 (treat negative float literals as static) #4063

Merged
merged 2 commits into from
Jun 23, 2023
Merged

bugfix: issue #4054 (treat negative float literals as static) #4063

merged 2 commits into from
Jun 23, 2023

Conversation

crusso
Copy link
Contributor

@crusso crusso commented Jun 23, 2023 •
edited
Loading

Fix for issue #4054
Adopting the same hack we use for integral literals.

@crusso crusso requested review from ggreif and nomeata June 23, 2023 14:09
@github-actions
Copy link

Comparing from 6cbf42c to 61450cd:
In terms of gas, no changes are observed in 4 tests.
In terms of size, no changes are observed in 4 tests.

@crusso crusso added the automerge-squash When ready, merge (using squash) label Jun 23, 2023
@mergify mergify bot merged commit 566defa into master Jun 23, 2023
@mergify mergify bot removed the automerge-squash When ready, merge (using squash) label Jun 23, 2023
@mergify mergify bot deleted the claudio/issue-4054 branch June 23, 2023 16:58
@crusso crusso restored the claudio/issue-4054 branch June 26, 2023 11:49
@crusso crusso deleted the claudio/issue-4054 branch June 26, 2023 11:55
@crusso crusso mentioned this pull request Jun 26, 2023
Merged
mergify bot pushed a commit that referenced this pull request Jun 26, 2023
@crusso
chore: update changelog (#4073)
33b481c 
Add changelog entry for #4063.

Fixed #4054
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nomeata nomeata nomeata approved these changes

@ggreif ggreif Awaiting requested review from ggreif

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@crusso @nomeata