Details
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
| libc/src/__support/FPUtil/PlatformDefs.h | ||
|---|---|---|
| 18–19 | 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–19 | 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?