Details
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
libc/src/__support/FPUtil/PlatformDefs.h | ||
---|---|---|
18 | Apologies for the drive-by comment. |
Thanks for catching the build problems with exhaustive tests. They have bit rotted a bit because we don't run them on any bot.
libc/src/__support/FPUtil/PlatformDefs.h | ||
---|---|---|
18 | On aarch64, long doubles are not the 64-bit IEEE double precision numbers. So, this would be incorrect after the typo is fixed. | |
libc/test/src/math/exhaustive/cosf_test.cpp | ||
12 | You should not need these headers to be included at all. |
Sorry for the delay, Louis was helping me upstream this but now I can update it directly.
LGTM, but is there any document that can be added as a reference in a comment in PlatformDefs.h?
Apologies for the drive-by comment.
I just noticed the spelling arch64. It looks like maybe this should be aarch64?