Skip to content

Long double versions of math functions#69

Merged
ViralBShah merged 2 commits intomasterfrom
vs/longdouble
Dec 4, 2014
Merged

Long double versions of math functions#69
ViralBShah merged 2 commits intomasterfrom
vs/longdouble

Conversation

@ViralBShah
Copy link
Copy Markdown
Member

Import long double versions of a bunch of stuff from openbsd.

Bump version number and SO major version, since we have
introduced new long double APIs.
ViralBShah added a commit that referenced this pull request Dec 4, 2014
Long double versions of math functions
@ViralBShah ViralBShah merged commit f876e59 into master Dec 4, 2014
@ViralBShah ViralBShah deleted the vs/longdouble branch December 4, 2014 18:29
@ViralBShah ViralBShah mentioned this pull request Dec 4, 2014
@staticfloat
Copy link
Copy Markdown
Contributor

This means we need to package a new version of openlibm, right? @nalimilan @sebastien-villemot

@nalimilan
Copy link
Copy Markdown
Contributor

@staticfloat Yes, but I'd like the test failure at #55 to be fixed or worked around first.

Comment thread Make.inc
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The last time I made the same mistake, @sebastien-villemot noted that we shouldn't raise the major version just because we added functions, but only when breaking compatibility for existing functions. Would you revert that change? Just raise SOMINOR instead.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Good point. Done.

@ViralBShah
Copy link
Copy Markdown
Member Author

We probably need to test this well and iron out issues. Since Julia does not use any of this yet, we do not need to release a new version anytime soon.

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.

3 participants