On Fri, Jun 27, 2025 at 03:38:32PM +0000, Yuao Ma wrote:
> Hi Jakub,
> 
> > Please don't include math.h here.
> 
> Done.
> 
> > And instead of this line use __builtin_acospi (0.5).
> > and, in dejagnu for runtime tests we prefer __builtin_abort on failure, so
> 
> Done.

Oh, one more thing.
signbit is documented to be a macro, so please don't declare
int signbit (double);
function in the testcase and instead of signbit use __builtin_signbit.

Ok for trunk with that fixed.

        Jakub

Reply via email to