Re: Please revert b2e843161d
- Reply: Andrew Turner : "Re: Please revert b2e843161d"
- In reply to: Andrew Turner : "Re: Please revert b2e843161d"
- Go to: [ bottom of page ] [ top of archives ] [ this month ]
Date: Thu, 25 Jun 2026 18:32:23 UTC
On 6/25/26 10:57, Andrew Turner wrote: > > I don’t expect it to work on amd64 in the general case. Godbolt shows > both GCC and Clang generating an infinite loop jumping to its self [1]. > It may work on some architecture levels, however I don’t know enough > about amd64 to know if it’s safe to enable. > > The USE_BUILTIN_* flags aren’t intended on being set by users as you found. The comment in the source code certainly does not alert a user to not use -DUSE_BUILTIN_FMAF (or any of the other USE_BUILTIN_*). Architecture specific code should be in msun/arm/. It would also seem to me that if this is indeed arm specific then it should be guarded with "#if defined(__ARM) && ..." -- steve