commit ad315ebfee626dbd472b9d68a76632f9e68bc8f5
author Mats Petersson <mats.petersson@arm.com> Tue Jul 09 10:35:06 2024 +0100
committer GitHub <noreply@github.com> Tue Jul 09 10:35:06 2024 +0100
tree bd055edaa25c3e5b3ce5b6a1c29ba86f3567f96b
parent 2b56005fd91de3adba1957e70575d5461329178b

[MLIR][NFC]Fix unusued variable warning. (#97813)

This just changes a dyn_cast to and isa call to check for type, which
avoids getting a warning in clang. The variable `ity` is not used in the
next block.

No functional change.

mlir/lib/Target/LLVMIR/ModuleTranslation.cpp

diff --git a/mlir/lib/Target/LLVMIR/ModuleTranslation.cpp b/mlir/lib/Target/LLVMIR/ModuleTranslation.cpp
index bc0f709..1d2e472 100644
--- a/mlir/lib/Target/LLVMIR/ModuleTranslation.cpp
+++ b/mlir/lib/Target/LLVMIR/ModuleTranslation.cpp
@@ -639,8 +639,7 @@
         if (llvm::ConstantDataSequential::isElementTypeCompatible(
                 elementType)) {
           // TODO: Handle all compatible types. This code only handles integer.
-          if (llvm::IntegerType *iTy =
-                  dyn_cast<llvm::IntegerType>(elementType)) {
+          if (isa<llvm::IntegerType>(elementType)) {
             if (llvm::ConstantInt *ci = dyn_cast<llvm::ConstantInt>(child)) {
               if (ci->getBitWidth() == 8) {
                 SmallVector<int8_t> constants(numElements, ci->getZExtValue());