[PATCH] Fix parameter type in C++ version of iseqsig (bug 23171)
Joseph Myers
joseph@codesourcery.com
Mon May 14 15:16:00 GMT 2018
On Mon, 14 May 2018, Gabriel F. T. Gomes wrote:
> On Mon, 14 May 2018, Gabriel F. T. Gomes wrote:
>
> >mistakenly used double parameters in the long double version of iseqsig,
> >thus causing spurious conversions to double, as reported on bug 23171.
>
> By the way, is there any kind of test, which could be added to the test
> suite, and that would have detected such spurious conversions?
I would guess something like
volatile long double a = LDBL_MAX;
volatile long double b = nextdownl (a);
/* Then test a and b do not compare equal with iseqsig. */
(not tested that this does indeed fail before and pass after the fix when
compiled as C++ on a platform where long double is wider than double).
--
Joseph S. Myers
joseph@codesourcery.com
More information about the Libc-alpha
mailing list