Hi! On Tue, Jun 05, 2018 at 02:02:05PM -0400, Michael Meissner wrote: > I tested this on a little endian power8 system running Linux. There were no > regressions. Can I install this in the trunk and eventually back port it to > GCC 8.2?
Please test on a BE system as well? And maybe on a p9? I don't expect problems, but we should know. (No need to test this patch, we'll find out after you commit in the unlikely case there are problems; buy in the future, please do). > --- gcc/config/rs6000/rs6000.c (revision 261170) > +++ gcc/config/rs6000/rs6000.c (working copy) > @@ -200,6 +200,17 @@ int dot_symbols; > of this machine mode. */ > scalar_int_mode rs6000_pmode; > > +/* Note whether IEEE 128-bit floating point was passed or returned, either as > + the __float128/_Float128 explicit type, or when long double is IEEE > 128-bit I think you mean __ieee128? There is no __float128 type anymore. > +static bool rs6000_passes_ieee128; > + > +/* Generate the manged name (i.e. U10__float128) used in GCC 8.1, and not the > + name used in current releases (i.e. u9__ieee128). */ > +static bool ieee128_mangling_gcc_8_1; It's a manged name, sure, but you mean mangled :-) Okay for trunk and backports. Thanks! Segher