cleanup

Author: V-FEXrt

tools/clang/unittests/HLSLExec/LinAlgTests.cpp

@@ -5,7 +5,7 @@
 // This file is distributed under the University of Illinois Open Source     //
 // License. See LICENSE.TXT for details.                                     //
 //                                                                           //
-// Execution tests for dx::linalg builtins (Proposal 0035, SM 6.10).        //
+// Execution tests for dx::linalg builtins                                   //
 //                                                                           //
 ///////////////////////////////////////////////////////////////////////////////
 
@@ -52,10 +52,6 @@ static int elemSize(ComponentType CT) {
   }
 }
 
-// ===========================================================================
-// ShaderOp construction helpers
-// ===========================================================================
-
 /// Create a ShaderOp for a compute shader dispatch.
 static std::unique_ptr<st::ShaderOp>
 createComputeOp(const char *Source, const char *Target, const char *RootSig,
@@ -129,14 +125,9 @@ runShaderOp(ID3D12Device *Device, dxc::SpecificDllLoader &DxcSupport,
       Device, DxcSupport, nullptr, std::move(InitCallback), std::move(OpSet));
 }
 
-// ===========================================================================
-// Shader compilation helper
-// ===========================================================================
-
 /// Compiles an HLSL shader using the DXC API to verify it is well-formed.
 /// This runs without a D3D12 device, so it works even when no SM 6.10
 /// hardware is available. Fails the test (via VERIFY) on compile error.
-
 static void compileShader(dxc::SpecificDllLoader &DxcSupport,
                           const char *Source, const char *Target,
                           const std::string &Args) {
@@ -190,10 +181,6 @@ static void compileShader(dxc::SpecificDllLoader &DxcSupport,
   }
 }
 
-// ===========================================================================
-// Test parameters
-// ===========================================================================
-
 struct MatrixParams {
   ComponentType CompType;
   int M;
@@ -217,10 +204,6 @@ struct MatrixParams {
   size_t totalBytes() const { return totalElements() * elemSize(CompType); }
 };
 
-// ===========================================================================
-// Compiler arguments builder
-// ===========================================================================
-
 static std::string buildCompilerArgs(const MatrixParams &Params, int Stride,
                                      const char *ExtraDefines = nullptr) {
   std::stringstream SS;
@@ -240,10 +223,6 @@ static std::string buildCompilerArgs(const MatrixParams &Params, int Stride,
   return SS.str();
 }
 
-// ===========================================================================
-// Verification helpers
-// ===========================================================================
-
 static bool verifyFloatBuffer(const void *Actual, const float *Expected,
                               size_t Count, bool Verbose,
                               float Tolerance = 0.0f) {
@@ -284,10 +263,6 @@ static bool verifyIntBuffer(const void *Actual, const int32_t *Expected,
   return Success;
 }
 
-// ===========================================================================
-// Test class
-// ===========================================================================
-
 class DxilConf_SM610_LinAlg {
 public:
   BEGIN_TEST_CLASS(DxilConf_SM610_LinAlg)
@@ -319,10 +294,6 @@ class DxilConf_SM610_LinAlg {
   std::optional<D3D12SDKSelector> D3D12SDK;
 };
 
-// ===========================================================================
-// Class setup
-// ===========================================================================
-
 bool DxilConf_SM610_LinAlg::setupClass() {
   WEX::TestExecution::SetVerifyOutput VerifySettings(
       WEX::TestExecution::VerifyOutputSettings::LogOnlyFailures);
@@ -357,7 +328,6 @@ bool DxilConf_SM610_LinAlg::setupClass() {
             L"test.");
         return false;
       }
-      // No device — tests will compile shaders and skip execution.
     }
   }
 
@@ -380,10 +350,6 @@ bool DxilConf_SM610_LinAlg::setupMethod() {
   return true;
 }
 
-// ===========================================================================
-// Load/Store roundtrip
-// ===========================================================================
-
 static const char LoadStoreShader[] = R"(
   RWByteAddressBuffer Input : register(u0);
   RWByteAddressBuffer Output : register(u1);
@@ -502,10 +468,6 @@ void DxilConf_SM610_LinAlg::LoadStoreRoundtrip_Wave_I32() {
   runLoadStoreRoundtrip(D3DDevice, DxcSupport, Params, VerboseLogging);
 }
 
-// ===========================================================================
-// Splat + Store
-// ===========================================================================
-
 static const char SplatStoreShader[] = R"(
   RWByteAddressBuffer Output : register(u0);