Skip to content

Support for Float literals #7

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

Merged
merged 8 commits into from
Mar 12, 2018
Merged

Conversation

mmroz
Copy link

@mmroz mmroz commented Jan 21, 2018

Adding basic support for float literals
Issues: #8 #9

Resolves SR-NNNN.

mmroz and others added 4 commits January 21, 2018 09:30
Handle Float Literal using BigDecimal
Accidentally made a type in the comment explaining Infinity and NaN, fixed that in this commit
@aoli-al
Copy link

aoli-al commented Feb 8, 2018

WALAWalker doesn't capture canonical SIL instructions and that's why FloatLiteralInstr always returns an 80bits float. I've fixed this via #18. Now you may want to modify this PR so that it is able to handle double and float separately.

@karimhamdanali
Copy link

Hi @themaplelab/swift , is that PR still relevant? I see that @Leeleo3x says that this should be modified after #18.

added check for float and double using if-statements
@bryantam
Copy link

This is still relevant. The modification has been just merged into this PR.

Copy link

@karimhamdanali karimhamdanali left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good to me

@karimhamdanali karimhamdanali merged commit a429fdb into wala Mar 12, 2018
@mmroz mmroz mentioned this pull request Apr 6, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants