[libc-commits] [PATCH] D153823: [libc][math] Fix floating-point test support on x86_64 Apple machines

Dominic Chen via Phabricator via libc-commits libc-commits at lists.llvm.org
Mon Jun 26 20:12:56 PDT 2023


ddcc added inline comments.


================
Comment at: libc/src/__support/FPUtil/x86_64/FEnvImpl.h:220
+  uint32_t mxcsr = internal::get_mxcsr();
   // Check both x87 status word and MXCSR.
+  uint16_t status_value = internal::get_status_value_for_except(excepts);
----------------
Just wanted to point out that this change will affect all x86_64 platforms. But based on the comment, I believe this function is supposed to have been fetching the exception status bits from bot hteh x87 FPU and the SSE unit anyway?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D153823/new/

https://reviews.llvm.org/D153823



More information about the libc-commits mailing list