Unverified Commit 74734782 authored by Aiden Grossman's avatar Aiden Grossman Committed by GitHub
Browse files

[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.
parent 0193af47
Loading
Loading
Loading
Loading
+2 −3
Original line number Diff line number Diff line
@@ -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);