Windows fenv_t does not include the MXCSR register and
the unused bits at the end of the x87 status. So we
exclude them in our struct definitions to make it
easy to read/write the state. getEnv and setEnv
were also excluded to avoid using MXCSR, but a
forthcoming patch will handle these functions.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
libc/utils/FPUtil/x86_64/FEnv.h | ||
---|---|---|
94 | according to https://docs.microsoft.com/en-us/cpp/preprocessor/predefined-macros, we don't need to check for _WIN64, _WIN32 is enough we should clean this up everywhere in libc |
Comment Actions
I think this is not correct but OK for now to unblock other work. We will revisit again when we enable fenv functions on Windows.
according to https://docs.microsoft.com/en-us/cpp/preprocessor/predefined-macros, we don't need to check for _WIN64, _WIN32 is enough
we should clean this up everywhere in libc