On Wed, 25 Oct 2017, Michael Meissner wrote: > +static const char *const fltfn_suffixes[] = { "F16", "F32", "F128", "F32X", > + "F64X", "F128X", NULL };
I'd expect this to include F64. If there's some reason that's inappropriate and the omission is deliberate, it needs a detailed comment explaining the omission. I don't think that, given the availability of fmaf128 etc. built-in functions with appropriate options, whether __FP_FAST_* are defined should actually depend on whether the user has passed options to disable those functions (after all, it doesn't for the existing fma / fmaf / fmal, and individual built-in functions can be disabled with -fno-builtin-<function> so the logic you have wouldn't work to detect whether the built-in function is disabled anyway). -- Joseph S. Myers jos...@codesourcery.com