[clang] 7473478 - [Clang][HLSL] Fix -Wunused-variable (#194374)

[via cfe-commits]

Author: Aiden Grossman
Date: 2026-04-27T06:52:28-07:00
New Revision: 74734782b42b640db1c6995e2d88388a5435fbfe

URL: https://github.com/llvm/llvm-project/commit/74734782b42b640db1c6995e2d88388a5435fbfe
DIFF: https://github.com/llvm/llvm-project/commit/74734782b42b640db1c6995e2d88388a5435fbfe.diff

LOG: [Clang][HLSL] Fix -Wunused-variable (#194374)

Inline the variable definition into the assert given it is side effect
free and the variable name does not make the code much more clear.

Added: 
    

Modified: 
    clang/lib/CodeGen/CGHLSLRuntime.cpp

Removed: 
    


################################################################################
diff  --git a/clang/lib/CodeGen/CGHLSLRuntime.cpp b/clang/lib/CodeGen/CGHLSLRuntime.cpp
index f2db4114c0798..a134f6aab9490 100644
--- a/clang/lib/CodeGen/CGHLSLRuntime.cpp
+++ b/clang/lib/CodeGen/CGHLSLRuntime.cpp
@@ -1658,10 +1658,9 @@ class HLSLBufferCopyEmitter {
                             llvm::ArrayType *DstTy) {
     // Those assumptions are checked by isBufferLayoutArray.
     auto *SrcPaddedArrayTy = cast<llvm::ArrayType>(SrcTy->getElementType(0));
-    auto *SrcPaddedEltTy =
-        cast<llvm::StructType>(SrcPaddedArrayTy->getElementType());
     assert(SrcPaddedArrayTy->getNumElements() + 1 == DstTy->getNumElements());
-    assert(SrcPaddedEltTy->getElementType(0) == SrcTy->getElementType(1));
+    assert(cast<llvm::StructType>(SrcPaddedArrayTy->getElementType())
+               ->getElementType(0) == SrcTy->getElementType(1));
 
     auto *SrcDataTy = SrcTy->getElementType(1);
     auto Zero = llvm::ConstantInt::get(CGF.IntTy, 0);