[bug] Execution Tests: Long Vector fix tolerances on trig functions (microsoft#7762)

This PR sets the proper tolerance for float16 trigonometric ops. It was
set to the wrong value by accident.

Author: alsepkow

tools/clang/unittests/HLSLExec/LongVectors.cpp

@@ -743,34 +743,6 @@ template <typename T> struct TrigonometricOperation {
   static T tanh(T Val) { return std::tanh(Val); }
 };
 
-template <typename T>
-void dispatchTrigonometricTest(const TestConfig &Config,
-                               ValidationConfig ValidationConfig,
-                               TrigonometricOpType OpType, size_t VectorSize) {
-#define DISPATCH(OP, NAME)                                                     \
-  case OP:                                                                     \
-    return dispatchUnaryTest<T>(Config, ValidationConfig, OP, VectorSize,      \
-                                TrigonometricOperation<T>::NAME, "")
-
-  switch (OpType) {
-    DISPATCH(TrigonometricOpType::Acos, acos);
-    DISPATCH(TrigonometricOpType::Asin, asin);
-    DISPATCH(TrigonometricOpType::Atan, atan);
-    DISPATCH(TrigonometricOpType::Cos, cos);
-    DISPATCH(TrigonometricOpType::Cosh, cosh);
-    DISPATCH(TrigonometricOpType::Sin, sin);
-    DISPATCH(TrigonometricOpType::Sinh, sinh);
-    DISPATCH(TrigonometricOpType::Tan, tan);
-    DISPATCH(TrigonometricOpType::Tanh, tanh);
-  case TrigonometricOpType::EnumValueCount:
-    break;
-  }
-
-#undef DISPATCH
-
-  LOG_ERROR_FMT_THROW(L"Unexpected TrigonometricOpType: %d.", OpType);
-}
-
 void dispatchTestByOpTypeAndVectorSize(const TestConfig &Config,
                                        TrigonometricOpType OpType,
                                        size_t VectorSize) {
@@ -780,18 +752,68 @@ void dispatchTestByOpTypeAndVectorSize(const TestConfig &Config,
   // as per the D3D functional specs. An example with this spec for sin and
   // cos is available here:
   // https://microsoft.github.io/DirectX-Specs/d3d/archive/D3D11_3_FunctionalSpec.htm#22.10.20
+  const ValidationConfig ValidationConfig = ValidationConfig::Epsilon(0.0008f);
+
+#define DISPATCH(TYPE, FUNC)                                                   \
+  if (Config.DataType == getDataTypeName<TYPE>())                              \
+    return dispatchUnaryTest(Config, ValidationConfig, OpType, VectorSize,     \
+                             TrigonometricOperation<TYPE>::FUNC, "");
+
+  switch (OpType) {
+  case TrigonometricOpType::Acos:
+    DISPATCH(HLSLHalf_t, acos);
+    DISPATCH(float, acos);
+    break;
+
+  case TrigonometricOpType::Asin:
+    DISPATCH(HLSLHalf_t, asin);
+    DISPATCH(float, asin);
+    break;
 
-  if (Config.DataType == getDataTypeName<HLSLHalf_t>())
-    return dispatchTrigonometricTest<HLSLHalf_t>(
-        Config, ValidationConfig::Epsilon(0.0010f), OpType, VectorSize);
+  case TrigonometricOpType::Atan:
+    DISPATCH(HLSLHalf_t, atan);
+    DISPATCH(float, atan);
+    break;
+
+  case TrigonometricOpType::Cos:
+    DISPATCH(HLSLHalf_t, cos);
+    DISPATCH(float, cos);
+    break;
+
+  case TrigonometricOpType::Cosh:
+    DISPATCH(HLSLHalf_t, cosh);
+    DISPATCH(float, cosh);
+    break;
 
-  if (Config.DataType == getDataTypeName<float>())
-    return dispatchTrigonometricTest<float>(
-        Config, ValidationConfig::Epsilon(0.0008f), OpType, VectorSize);
+  case TrigonometricOpType::Sin:
+    DISPATCH(HLSLHalf_t, sin);
+    DISPATCH(float, sin);
+    break;
+
+  case TrigonometricOpType::Sinh:
+    DISPATCH(HLSLHalf_t, sinh);
+    DISPATCH(float, sinh);
+    break;
+
+  case TrigonometricOpType::Tan:
+    DISPATCH(HLSLHalf_t, tan);
+    DISPATCH(float, tan);
+    break;
+
+  case TrigonometricOpType::Tanh:
+    DISPATCH(HLSLHalf_t, tanh);
+    DISPATCH(float, tanh);
+    break;
+
+  case TrigonometricOpType::EnumValueCount:
+    break;
+  }
+
+#undef DISPATCH
 
   LOG_ERROR_FMT_THROW(
-      L"DataType '%s' not supported for trigonometric operations.",
-      (const wchar_t *)Config.DataType);
+      L"DataType '%s' not supported for TrigonometricOpType'%s'",
+      (const wchar_t *)Config.DataType, (const wchar_t *)Config.OpTypeEnum);
 }
 
 //