[clang] [HLSL] Implement the `lit` intrinsic (PR #134171)

Farzon Lotfi via cfe-commits cfe-commits at lists.llvm.org
Thu Apr 3 14:48:10 PDT 2025


================
@@ -253,6 +253,37 @@ const inline float length(__detail::HLSL_FIXED_VECTOR<float, N> X) {
   return __detail::length_vec_impl(X);
 }
 
+//===----------------------------------------------------------------------===//
+// lit builtins
+//===----------------------------------------------------------------------===//
+
+/// \fn vector<T, 4> lit(T NDotL, T NDotH, T M)
+/// \brief Returns a lighting coefficient vector.
+/// \param NDotL The dot product of the normalized surface normal and the
+/// light vector.
+/// \param NDotH The dot product of the half-angle vector and the surface
+/// normal.
+/// \param M A specular exponent.
+///
+/// This function returns a lighting coefficient vector (ambient, diffuse,
+/// specular, 1).
+
+template <typename T>
+_HLSL_16BIT_AVAILABILITY(shadermodel, 6.2)
+const inline __detail::enable_if_t<__detail::is_arithmetic<T>::Value &&
----------------
farzonl wrote:

> @farzonl The template is needed for the `__detail::is_arithmetic<T>` check since `lit` is only supposed to accept scalars as inputs. Without the check, it would allow types like float4 as input as it would be implicitly truncated to a float.

I understand that logic makes sense. That said i think the decision should fall to what DXC does. If DXC doesn’t implicit cast then i think we should aswell.

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


More information about the cfe-commits mailing list