clean up

Author: joaosaffran

tools/clang/unittests/HLSLExec/LongVectorOps.def

@@ -134,7 +134,6 @@ OP_DEFAULT(UnaryMath, Log10, 1, "log10", "")
 OP_DEFAULT(UnaryMath, Log2, 1, "log2", "")
 OP_DEFAULT_DEFINES(UnaryMath, Frexp, 1, "TestFrexp", "", " -DFUNC_FREXP=1")
 
-
 OP_DEFAULT(FloatSpecial, IsFinite, 1, "isfinite", "")
 OP_DEFAULT(FloatSpecial, IsInf, 1, "isinf", "")
 OP_DEFAULT(FloatSpecial, IsNan, 1, "isnan", "")

tools/clang/unittests/HLSLExec/LongVectors.cpp

@@ -786,7 +786,7 @@ struct ExpectedBuilder<OpType::ArrayOperator_SingleAccess, T> {
 };
 
 template <typename T>
-static std::vector<T> BuildLoop2(const InputSets<T> &Inputs) {
+static std::vector<T> buildExpectedArrayAccessOutput(const InputSets<T> &Inputs) {
   DXASSERT_NOMSG(Inputs.size() == 2);
   const size_t VectorSize = Inputs[0].size();
   std::vector<T> Expected;
@@ -807,7 +807,7 @@ struct Op<OpType::ArrayOperator_Loop, T, 2> : DefaultValidation<T> {};
 template <typename T> struct ExpectedBuilder<OpType::ArrayOperator_Loop, T> {
   static std::vector<T> buildExpected(Op<OpType::ArrayOperator_Loop, T, 2>,
                                       const InputSets<T> &Inputs) {
-    return BuildLoop2(Inputs);
+    return buildExpectedArrayAccessOutput(Inputs);
   }
 };
 
@@ -817,7 +817,7 @@ struct Op<OpType::ArrayOperator_Unroll, T, 2> : DefaultValidation<T> {};
 template <typename T> struct ExpectedBuilder<OpType::ArrayOperator_Unroll, T> {
   static std::vector<T> buildExpected(Op<OpType::ArrayOperator_Unroll, T, 2>,
                                       const InputSets<T> &Inputs) {
-    return BuildLoop2(Inputs);
+    return buildExpectedArrayAccessOutput(Inputs);
   }
 };
 

tools/clang/unittests/HLSLExec/ShaderOpArith.xml

@@ -4102,7 +4102,7 @@ void MSMain(uint GID : SV_GroupIndex,
         #endif
 
         [numthreads(1,1,1)]
-        void main(uint GI : SV_GroupIndex, uint3 dispatchThreadID : SV_DispatchThreadID) {
+        void main(uint GI : SV_GroupIndex) {
 
           #ifdef FUNC_SHUFFLE_VECTOR
             // For shuffle vector, the input is a scalar, not a vector.