[clang] [clang][docs] Revise documentation for `__builtin_reduce_(max|min)`. (PR #114637)

via cfe-commits cfe-commits at lists.llvm.org
Tue Nov 5 16:53:24 PST 2024


================
@@ -745,12 +745,8 @@ Let ``VT`` be a vector type and ``ET`` the element type of ``VT``.
 ======================================= ====================================================================== ==================================
          Name                            Operation                                                              Supported element types
 ======================================= ====================================================================== ==================================
- ET __builtin_reduce_max(VT a)           return x or y, whichever is larger; If exactly one argument is         integer and floating point types
-                                         a NaN, return the other argument. If both arguments are NaNs,
-                                         fmax() return a NaN.
- ET __builtin_reduce_min(VT a)           return x or y, whichever is smaller; If exactly one argument           integer and floating point types
-                                         is a NaN, return the other argument. If both arguments are
-                                         NaNs, fmax() return a NaN.
+ ET __builtin_reduce_max(VT a)           return the largest element of the vector.                              integer and floating point types
+ ET __builtin_reduce_min(VT a)           return the smallest element of the vector.                             integer and floating point types
----------------
c8ef wrote:

I'm not sure if we had any conformance testing before, but since this is just a document change and the patch is keeping the clang builtin document aligned with the llvm intrinsic document, maybe there's no need to add extra tests here?

https://github.com/llvm/llvm-project/pull/114637


More information about the cfe-commits mailing list