On Wednesday 28 February 2024 23:24:25 Martin Storsjö wrote:
> On Wed, 28 Feb 2024, Pali Rohár wrote:
>
> > All commented lines refers to non-underscore aliases for underscored
> > variant of functions. Aliases for some reasons are not prevent, e.g.
> > because some aliases are defined in other files (CRT_fp10.c and CRT_fp8.c).
>
> I don't understand what the second sentence here is saying, can you rephrase
> it?
Ok, I will try to explain it a bit more. ADD_UNDERSCORE is a macro which
just adds a symbol alias. It does not export any new symbol.
#define ADD_UNDERSCORE(symbol) symbol == _ ## symbol
Symbol "fpreset" is already defined in CRT_fp10.c and CRT_fp8.c files as
an alias to the "_fpreset" symbol.
So ADD_UNDERSCORE(fpreset) should not be added into msvcrt-common.def.in
because same job is already done by __attribute__((alias("_fpreset"))).
> > ---
> > mingw-w64-crt/def-include/msvcrt-common.def.in | 15 ++++++++-------
> > 1 file changed, 8 insertions(+), 7 deletions(-)
> >
>
> > @@ -134,13 +135,13 @@ ADD_UNDERSCORE(y0)
> > ADD_UNDERSCORE(y1)
> > ADD_UNDERSCORE(yn)
> > ADD_UNDERSCORE(chgsign)
> > -;scalb
> > +ADD_UNDERSCORE(scalb)
>
> This change here is, accidentally?, actually adding the alias even though it
> was supposed to be commented out?
>
> // Martin
Ah, it should not be there. My attempt was to cleanup comments without
any functional change. But all those parenthesis, semicolons and pluses
in diff make me hard to miss some accidental changes.
_______________________________________________
Mingw-w64-public mailing list
Mingw-w64-public@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/mingw-w64-public
