* [musl] [PATCH] math/x32: correct lrintl.s for 32-bit long
@ 2020-01-18 16:15 Alexander Monakov
2020-01-18 16:51 ` Szabolcs Nagy
2020-01-25 16:38 ` Alexander Monakov
0 siblings, 2 replies; 4+ messages in thread
From: Alexander Monakov @ 2020-01-18 16:15 UTC (permalink / raw)
To: musl
[-- Attachment #1: Type: text/plain, Size: 247 bytes --]
---
Looks like the implementation copied from x86_64 was not adjusted for
32-bit 'long' on x32, but apart from failure to raise FE_INVALID this
error was harmless.
src/math/x32/lrintl.s | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0320-math-x32-correct-lrintl.s-for-32-bit-long.patch --]
[-- Type: text/x-patch; name="0320-math-x32-correct-lrintl.s-for-32-bit-long.patch", Size: 291 bytes --]
diff --git a/src/math/x32/lrintl.s b/src/math/x32/lrintl.s
index ee97d1cf..d4355c32 100644
--- a/src/math/x32/lrintl.s
+++ b/src/math/x32/lrintl.s
@@ -2,6 +2,6 @@
.type lrintl,@function
lrintl:
fldt 8(%esp)
- fistpll 8(%esp)
- mov 8(%esp),%rax
+ fistpl 8(%esp)
+ movl 8(%esp),%eax
ret
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [musl] [PATCH] math/x32: correct lrintl.s for 32-bit long
2020-01-18 16:15 [musl] [PATCH] math/x32: correct lrintl.s for 32-bit long Alexander Monakov
@ 2020-01-18 16:51 ` Szabolcs Nagy
2020-01-25 16:38 ` Alexander Monakov
1 sibling, 0 replies; 4+ messages in thread
From: Szabolcs Nagy @ 2020-01-18 16:51 UTC (permalink / raw)
To: musl
* Alexander Monakov <amonakov@ispras.ru> [2020-01-18 19:15:16 +0300]:
> ---
> Looks like the implementation copied from x86_64 was not adjusted for
> 32-bit 'long' on x32, but apart from failure to raise FE_INVALID this
> error was harmless.
fenv behaviour is wrong on out-of-range values, not just
because of missing invalid, but spurious inexact too.
thanks for finding this.
>
> src/math/x32/lrintl.s | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/src/math/x32/lrintl.s b/src/math/x32/lrintl.s
> index ee97d1cf..d4355c32 100644
> --- a/src/math/x32/lrintl.s
> +++ b/src/math/x32/lrintl.s
> @@ -2,6 +2,6 @@
> .type lrintl,@function
> lrintl:
> fldt 8(%esp)
> - fistpll 8(%esp)
> - mov 8(%esp),%rax
> + fistpl 8(%esp)
> + movl 8(%esp),%eax
> ret
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [musl] [PATCH] math/x32: correct lrintl.s for 32-bit long
2020-01-18 16:15 [musl] [PATCH] math/x32: correct lrintl.s for 32-bit long Alexander Monakov
2020-01-18 16:51 ` Szabolcs Nagy
@ 2020-01-25 16:38 ` Alexander Monakov
2020-01-27 18:03 ` Rich Felker
1 sibling, 1 reply; 4+ messages in thread
From: Alexander Monakov @ 2020-01-25 16:38 UTC (permalink / raw)
To: musl
Ping, doesn't appear to be applied yet.
On Sat, 18 Jan 2020, Alexander Monakov wrote:
> ---
> Looks like the implementation copied from x86_64 was not adjusted for
> 32-bit 'long' on x32, but apart from failure to raise FE_INVALID this
> error was harmless.
>
> src/math/x32/lrintl.s | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [musl] [PATCH] math/x32: correct lrintl.s for 32-bit long
2020-01-25 16:38 ` Alexander Monakov
@ 2020-01-27 18:03 ` Rich Felker
0 siblings, 0 replies; 4+ messages in thread
From: Rich Felker @ 2020-01-27 18:03 UTC (permalink / raw)
To: musl
On Sat, Jan 25, 2020 at 07:38:07PM +0300, Alexander Monakov wrote:
> Ping, doesn't appear to be applied yet.
>
> On Sat, 18 Jan 2020, Alexander Monakov wrote:
>
> > ---
> > Looks like the implementation copied from x86_64 was not adjusted for
> > 32-bit 'long' on x32, but apart from failure to raise FE_INVALID this
> > error was harmless.
> >
> > src/math/x32/lrintl.s | 4 ++--
> > 1 file changed, 2 insertions(+), 2 deletions(-)
> >
> >
Thanks for pinging! I'd missed that this was something that needed to
be applied now vs the other x86 math patches that are to be applied
after release. Applying now.
Rich
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2020-01-27 18:03 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-01-18 16:15 [musl] [PATCH] math/x32: correct lrintl.s for 32-bit long Alexander Monakov
2020-01-18 16:51 ` Szabolcs Nagy
2020-01-25 16:38 ` Alexander Monakov
2020-01-27 18:03 ` Rich Felker
Code repositories for project(s) associated with this public inbox
https://git.vuxu.org/mirror/musl/
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).