| Message ID | cb720e2b-f617-47a5-50ca-5c751ec6fc33@linux.vnet.ibm.com |
|---|---|
| State | New |
| Headers | show |
On Tue, 4 Oct 2016, Stefan Liebler wrote: > Hi, > > I've regenerated ulps file after recent commit "Use __builtin_fma more in > dbl-64 code.". > Ok to commit? You're an s390 maintainer. While subsystem maintainers can of course seek review of their own patches if they think it useful in a particular case, this sort of patch is expected just to be committed. The libm-test machinery ensures that excessive ulps will not be added and will be ignored if they get added.
On 10/04/2016 02:13 PM, Joseph Myers wrote: > On Tue, 4 Oct 2016, Stefan Liebler wrote: > >> Hi, >> >> I've regenerated ulps file after recent commit "Use __builtin_fma more in >> dbl-64 code.". >> Ok to commit? > > You're an s390 maintainer. While subsystem maintainers can of course seek > review of their own patches if they think it useful in a particular case, > this sort of patch is expected just to be committed. The libm-test > machinery ensures that excessive ulps will not be added and will be > ignored if they get added. > Okay. I've committed it. Thanks.
diff --git a/sysdeps/s390/fpu/libm-test-ulps b/sysdeps/s390/fpu/libm-test-ulps index bc5795b..9a72be0 100644 --- a/sysdeps/s390/fpu/libm-test-ulps +++ b/sysdeps/s390/fpu/libm-test-ulps @@ -126,9 +126,7 @@ ildouble: 4 ldouble: 4 Function: "atan": -double: 1 float: 1 -idouble: 1 ifloat: 1 ildouble: 1 ldouble: 1 @@ -372,9 +370,7 @@ ildouble: 3 ldouble: 3 Function: "carg": -double: 1 float: 1 -idouble: 1 ifloat: 1 ildouble: 2 ldouble: 2 @@ -884,9 +880,7 @@ ildouble: 2 ldouble: 2 Function: Imaginary part of "clog": -double: 1 float: 1 -idouble: 1 ifloat: 1 ildouble: 1 ldouble: 1 @@ -900,9 +894,9 @@ ildouble: 2 ldouble: 2 Function: Imaginary part of "clog10": -double: 1 +double: 2 float: 2 -idouble: 1 +idouble: 2 ifloat: 2 ildouble: 2 ldouble: 2