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.

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

Reenable quad and double precision in libm #545

Merged
merged 3 commits into from
Jun 5, 2024

Conversation

joanaxcruz
Copy link
Contributor

@joanaxcruz joanaxcruz commented May 30, 2024

In the latest release (3.6.0), there were some unexpected ABI changes as pointed out in #534.
We try to restore the functions that were lost by reenabling quad and double precision in libm.

blapie added a commit to blapie/sleef that referenced this pull request May 31, 2024
Add a new workflow to detect unexpected backward-incompatible
changes at precommit, by comparing current version against a
reference version, both generated on Linux with gcc.
It uses abidiff to compare ELF-format shared libraries.
This is the tool used by Fedora to report a recent breach in
API shibatch#534. Test should fail until it is fixed by shibatch#545.
blapie added a commit to blapie/sleef that referenced this pull request May 31, 2024
Add a new workflow to detect unexpected backward-incompatible
changes at precommit, by comparing current version against a
reference version, both generated on Linux with gcc.
It uses abidiff to compare ELF-format shared libraries.
This is the tool used by Fedora to report a recent breach in
API shibatch#534. Test should fail until it is fixed by shibatch#545.

Change-Id: I2e4efa4682e05d351535cf66cfc4adc806f6a67b
blapie added a commit to blapie/sleef that referenced this pull request May 31, 2024
Add a new workflow to detect unexpected backward-incompatible
changes at precommit, by comparing current version against a
reference version, both generated on Linux with gcc.
It uses abidiff to compare ELF-format shared libraries.
This is the tool used by Fedora to report a recent breach in
API shibatch#534. Test should fail until it is fixed by shibatch#545.
blapie added a commit to blapie/sleef that referenced this pull request May 31, 2024
Add a new workflow to detect unexpected backward-incompatible
changes at precommit, by comparing current version against a
reference version, both generated on Linux with gcc.
It uses abidiff to compare ELF-format shared libraries.
This is the tool used by Fedora to report a recent breach in
API shibatch#534. Test should fail until it is fixed by shibatch#545.
blapie added a commit to blapie/sleef that referenced this pull request May 31, 2024
Add a new workflow to detect unexpected backward-incompatible
changes at precommit, by comparing current version against a
reference version, both generated on Linux with gcc.
It uses abidiff to compare ELF-format shared libraries.
This is the tool used by Fedora to report a recent breach in
API shibatch#534. Test should fail until it is fixed by shibatch#545.
blapie added a commit to blapie/sleef that referenced this pull request May 31, 2024
Add a new workflow to detect unexpected backward-incompatible
changes at precommit, by comparing current version against a
reference version, both generated on Linux with gcc.
It uses abidiff to compare ELF-format shared libraries.
This is the tool used by Fedora to report a recent breach in
API shibatch#534. Test should fail until it is fixed by shibatch#545.
blapie added a commit to blapie/sleef that referenced this pull request May 31, 2024
Add a new workflow to detect unexpected backward-incompatible
changes at precommit, by comparing current version against a
reference version, both generated on Linux with gcc.
It uses abidiff to compare ELF-format shared libraries.
This is the tool used by Fedora to report a recent breach in
API shibatch#534. Test should fail until it is fixed by shibatch#545.
blapie added a commit to blapie/sleef that referenced this pull request May 31, 2024
Add a new workflow to detect unexpected backward-incompatible
changes at precommit, by comparing current version against a
reference version, both generated on Linux with gcc.
It uses abidiff to compare ELF-format shared libraries.
This is the tool used by Fedora to report a recent breach in
API shibatch#534. Test should fail until it is fixed by shibatch#545.
blapie added a commit to blapie/sleef that referenced this pull request May 31, 2024
Add a new workflow to detect unexpected backward-incompatible
changes at precommit, by comparing current version against a
reference version, both generated on Linux with gcc.
It uses abidiff to compare ELF-format shared libraries.
This is the tool used by Fedora to report a recent breach in
API shibatch#534. Test should fail until it is fixed by shibatch#545.
@joanaxcruz joanaxcruz force-pushed the joaxcru/reenable_quad_double branch from e10f191 to f485670 Compare June 5, 2024 08:33
In the transition between sleef-3.5 and sleef-3.6, support
for quad and long precision was dropped.

This resulted in 4 funtions being removed from the ABI:

    [D] 'function const Sleef_longdouble2 Sleef_sincospil_u05(long double)'    {Sleef_sincospil_u05}
    [D] 'function const Sleef_longdouble2 Sleef_sincospil_u35(long double)'    {Sleef_sincospil_u35}
    [D] 'function const Sleef_quad2 Sleef_sincospiq_u05(Sleef_quad)'    {Sleef_sincospiq_u05}
    [D] 'function const Sleef_quad2 Sleef_sincospiq_u35(Sleef_quad)'    {Sleef_sincospiq_u35}

By reverting these changes and bring these functions back to the ABI.

This reverts commit 64c957f.
Completed definition for Sleef_quad in `quaddef.h` where it can be
used by sleefqp.c
Note quad precision is not available for aarch64, as it does
not support quadmath library.
@joanaxcruz joanaxcruz force-pushed the joaxcru/reenable_quad_double branch from f485670 to da8e42c Compare June 5, 2024 08:37
@joanaxcruz joanaxcruz marked this pull request as ready for review June 5, 2024 12:24
@joanaxcruz joanaxcruz marked this pull request as draft June 5, 2024 12:24
@blapie blapie marked this pull request as ready for review June 5, 2024 12:41
@blapie blapie linked an issue Jun 5, 2024 that may be closed by this pull request
@blapie blapie merged commit db4f198 into shibatch:master Jun 5, 2024
43 checks passed
blapie added a commit that referenced this pull request Oct 9, 2024
Add a new workflow to detect unexpected backward-incompatible
changes at precommit, by comparing current version against a
reference version, both generated on Linux with gcc.
It uses abidiff to compare ELF-format shared libraries.
This is the tool used by Fedora to report a recent breach in
API #534. Test should fail until it is fixed by #545.
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.

A few API functions were (accidentally?) removed in 3.6
2 participants