Prevent clamp logic on bools

Author: alsepkow

tools/clang/unittests/HLSLExec/ExecutionTest.cpp

@@ -12179,6 +12179,14 @@ void ExecutionTest::LongVectorOpTestBase(
   T ClampArgC = 0;
   T ClampArgT = 0;
   if (TestConfig.OpType == LongVectorOpType_Clamp) {
+    if constexpr (std::is_same_v<T, HLSLBool_t>) {
+      // Attempting to generate a clamp value for HLSLBool_t will result in an
+      // infinite loop in the below while. We don't have a test case for clamp
+      // with bools anyways. But adding this check to prevent the mistake.
+      LogErrorFmtThrow(
+          L"Clamp is not supported for HLSLBool_t.");
+    }
+
     ClampArgC = NumberGenerator.generate();
     ClampArgT = NumberGenerator.generate();
     while (ClampArgC >= ClampArgT) {
@@ -12198,17 +12206,16 @@ void ExecutionTest::LongVectorOpTestBase(
       } else if (TestConfig.OperatorString == "+") {
         ExpectedVector[Index] = Input1 + Input2;
       } else if (TestConfig.OperatorString == ",") {
-        if (TestConfig.OpType == LongVectorOpType_Min)
+        if (TestConfig.OpType == LongVectorOpType_Min )
           ExpectedVector[Index] = std::min<T>(Input1, Input2);
         else if (TestConfig.OpType == LongVectorOpType_Max)
           ExpectedVector[Index] = std::max<T>(Input1, Input2);
         else
-          LogErrorFmtThrow(L"Unrecognized Binary LongVectorOpType: %d",
-                           TestConfig.OpType);
+          LogErrorFmtThrow(L"Unrecognized Binary LongVectorOpType: %d",TestConfig.OpType);
       } else {
         LogErrorFmtThrow(
-            L"Don't know how to compute expected value for operatorString: %s",
-            TestConfig.OperatorString.c_str());
+          L"Don't know how to compute expected value for operatorString: %s",
+          TestConfig.OperatorString.c_str());
       }
     } else // Unary op logic
     {
@@ -12218,8 +12225,7 @@ void ExecutionTest::LongVectorOpTestBase(
       } else if (TestConfig.OpType = LongVectorOpType_Initialize) {
         ExpectedVector[Index] = InputVector1[Index];
       } else {
-        LogErrorFmtThrow(L"Unrecognized Unary LongVectorOpType: %d",
-                         TestConfig.OpType);
+        LogErrorFmtThrow(L"Unrecognized Unary LongVectorOpType: %d",TestConfig.OpType);
       }
     }
   }