[clang] [Headers][X86] Add rounding and exception notes to conversions (PR #83447)
Paul T Robinson via cfe-commits
cfe-commits at lists.llvm.org
Fri Mar 8 08:22:25 PST 2024
================
@@ -1405,11 +1413,12 @@ static __inline__ __m128d __DEFAULT_FN_ATTRS _mm_cvtss_sd(__m128d __a,
/// Converts the two double-precision floating-point elements of a
/// 128-bit vector of [2 x double] into two signed 32-bit integer values,
-/// returned in the lower 64 bits of a 128-bit vector of [4 x i32].
+/// returned in the lower 64 bits of a 128-bit vector of [4 x i32],
+/// truncating inexact results.
----------------
pogo59 wrote:
Truncation is what distinguishes this from `_mm_cvtpd_epi32` so I think it is necessary? That information used to be in the second paragraph, I shortened it and moved it up to the first paragraph, to make that difference more obvious.
Behavior on overflow was added to the second paragraph.
Do you have a different suggestion for how to organize this information?
https://github.com/llvm/llvm-project/pull/83447
More information about the cfe-commits
mailing list