From mboxrd@z Thu Jan 1 00:00:00 1970 X-Spam-Checker-Version: SpamAssassin 3.4.4 (2020-01-24) on inbox.vuxu.org X-Spam-Level: X-Spam-Status: No, score=-1.0 required=5.0 tests=MAILING_LIST_MULTI, RCVD_IN_MSPIKE_H2 autolearn=ham autolearn_force=no version=3.4.4 Received: (qmail 26241 invoked from network); 11 Oct 2022 17:47:12 -0000 Received: from second.openwall.net (193.110.157.125) by inbox.vuxu.org with ESMTPUTF8; 11 Oct 2022 17:47:12 -0000 Received: (qmail 23900 invoked by uid 550); 11 Oct 2022 17:47:01 -0000 Mailing-List: contact musl-help@lists.openwall.com; run by ezmlm Precedence: bulk List-Post: List-Help: List-Unsubscribe: List-Subscribe: List-ID: Reply-To: musl@lists.openwall.com Received: (qmail 23859 invoked from network); 11 Oct 2022 17:47:00 -0000 Date: Tue, 11 Oct 2022 13:46:54 -0400 From: Rich Felker To: Markus Wichmann , musl@lists.openwall.com Message-ID: <20221011174654.GG29905@brightrain.aerifal.cx> References: <20221009084508.GB2442@voyager> <20221009103550.GJ2158779@port70.net> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20221009103550.GJ2158779@port70.net> User-Agent: Mutt/1.5.21 (2010-09-15) Subject: Re: [musl] Why thousands of digits in floatscan.c? On Sun, Oct 09, 2022 at 12:35:50PM +0200, Szabolcs Nagy wrote: > * Markus Wichmann [2022-10-09 10:45:08 +0200]: > > Hi all, > > > > so I recently read this wonderful paper: https://dl.acm.org/doi/10.1145/103162.103163 > > > > Great read, I encourage it for everyone, even though some things are not > > directly applicable to the C application programmer. > > > > Anyway, towards the end of the paper, Goldberg makes the argument that a > > 9 digit decimal number is always enough to uniquely identify a > > single-precision number. Generalizing out from his argument, I > > calculated that 36 decimal digits are needed for a quad-precision number > > (the digit count for a w bit mantissa is roundup(w * lg 2) + 1, which is > > 9 for single-prec, 17 for double-prec, 21 for double-extended, and 33 > > for double-double). > > this is {FLT,DBL,LDBL}_DECIMAL_DIG > > > > > Now, I looked again at floatscan.c. The algorithm in decfloat() mostly > > consists of reading all the decimal digits into a base 1 billion number, > > which could have any length at all, then aligning the radix point with a > > word boundary, and then shifting that number such that exactly one > > mantissa worth of bits are left of the radix point. For the most part, > > the algorithm does not care about the remaining bits except for two bits > > of information: If the tail is equal to 0, or else how it compares to > > 0.5. So now I am left wondering why then, if only the first > > LDBL_MANT_DIG bits of the number, plus two more for the tail, ever count > > for anything, the algorithm makes room for thousands of decimal digits. > > Would it not be enough to save a few dozen? What purpose do the far-back > > digits have? > > *_DECIMAL_DIG is enough if we know the digits are the decimal > conversion of an actual fp number of the right type and right > rounding mode. > > but e.g. it can be the digits of a number halfway between two fp > numbers exactly. that sequence of digits would never occur when > an actual fp number is printed but it is still a valid input and > has to be read to the end to decide which way to round. (assuming > nearest rounding mode.) > > the worst case is halfway between {FLT,DBL,LDBL}_TRUE_MIN and 0. > and the digits of that would be > > 2^{-150,-1075,-16495} = 5^{150,..} * 10^{-150,..} > > i.e. 0. followed by many zeros and then the digits of 5^16495. > depending on the last digit it has to be rounded up or down. Yep, this. Also, AFAICT it's impossible to tell before you know the exponent at what point you might be able to discard further mantissa digits and still get the right answer. There may be some shortcuts for this but it's non-obvious, and if you need to buffer all the mantissa digits anyway, the b1b representation is about the most efficient you can make it. Further, musl's floatscan raises FE_INEXACT exactly when the conversion from a decimal string to binary floating point is inexact. I don't think this is a requirement but it's a desirable property and comes nearly free if you do the above right. Rich