On Fri, Jan 15, 2021 at 03:43:13PM -0600, Segher Boessenkool wrote: > Hi! > > On Thu, Jan 14, 2021 at 11:59:19AM -0500, Michael Meissner wrote: > > >From 78435dee177447080434cdc08fc76b1029c7f576 Mon Sep 17 00:00:00 2001 > > From: Michael Meissner <meiss...@linux.ibm.com> > > Date: Wed, 13 Jan 2021 21:47:03 -0500 > > Subject: [PATCH] PowerPC: Map IEEE 128-bit long double built-ins. > > > > This patch replaces patches previously submitted: > > What did you change after I approved it?
You grumbled about the way I converted the names from the current name to the IEEE 128-bit name as being unclear. 1) I moved the table of known mappings from within a function to a separate function, and I populated the switch statement with all of the current names. 2) I moved the code that looks at a built-in function's arguments and returns whether it uses long double to a separate function rather than being buried within a larger function. 3) I changed the code for case we we didn't provide a name (i.e. new built-ins) to hopefully be clearer on the conversion. -- Michael Meissner, IBM IBM, M/S 2506R, 550 King Street, Littleton, MA 01460-6245, USA email: meiss...@linux.ibm.com, phone: +1 (978) 899-4797
