Hi,
AFAICS the code in build_round_expr implicitly assumes that __float128 exists,
which is *not* the common case among 64-bit architectures since "long double"
is generally already 128-bit for them.
Tested on x86-64/Linux and SPARC64/Linux, OK for the mainline?
2021-03-08 Eri
);
> fn = gfc_builtin_decl_for_float_kind (BUILT_IN_ROUND, kind);
> }
> else
Yes, it works fine on x86-64/Linux and SPARC64/Linux, applied, thanks.
--
Eric Botcazou
What do we do for other similar flags, e.g. TREE_READONLY?
--
Eric Botcazou
when the FIELD_DECL has
> TREE_THIS_VOLATILE set.
This would be weird semantics in my opinion.
> I guess I'll do one more experiment and add verification that
> TREE_THIS_VOLATILE on COMPONENT_REFs and FIELD_DECLs is consistent
> and see where that trips.
Sounds good to me.
--
Eric Botcazou
> So I'm leaning towards leaving build3 alone and fixing up frontends
> as issues pop up.
FWIW fine with me.
--
Eric Botcazou
> fixed now.
> bootstrapped successfully!
Thanks for fixing it. Another way out is to hide the Win32 API by defining
__GTHREAD_HIDE_WIN32API like libstdc++ does in its header files.
--
Eric Botcazou
