On 6.02.2025 13:42, Martin Storsjö wrote:
So would you feel ok with doing sincos() here with __MINGW_ASM_CALL(),
as practical Clang doesn't seem to break it at the moment, even though
it in theory could do that?
It seems to me that regardless of the solution here, it would be
interesting to make Clang stop optimizing to sincos() for mingw. If
we're going to expand it back to sin() and cos() in crt, it's
counter-productive to "optimize" sin()+cos() like that in the first
place. I drafted a patch for that [1], do you think I should submit it
upsteram? I know that your change affects only UCRT, but I suspect that
with msvcrt.dll the result may be similar and I think we should treat
UCRT as a primary target anyway.
With a change like that, we'd guarantee that future versions will not
"optimize" a C implementation in mingw-w64-crt.
Thanks,
Jacek
[1]
https://github.com/cjacek/llvm-project/commit/9823fe9c2e79ad337ad80ba8f706434c28778955
_______________________________________________
Mingw-w64-public mailing list
Mingw-w64-public@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/mingw-w64-public
