[X86] getShape - don't dereference dyn_cast<>
authorSimon Pilgrim <llvm-dev@redking.me.uk>
Thu, 26 Aug 2021 12:17:27 +0000 (13:17 +0100)
committerSimon Pilgrim <llvm-dev@redking.me.uk>
Thu, 26 Aug 2021 14:08:13 +0000 (15:08 +0100)
dyn_cast can return nullptr, use cast<> to assert we have the correct type.

llvm/lib/Target/X86/X86LowerAMXType.cpp

index e150f2d..29f7c90 100644 (file)
@@ -126,7 +126,7 @@ static std::pair<Value *, Value *> getShape(IntrinsicInst *II, unsigned OpNo) {
     case 5:
       if (isa<ConstantInt>(II->getArgOperand(2)))
         Row = Builder.getInt16(
-            (dyn_cast<ConstantInt>(II->getOperand(2))->getSExtValue()) / 4);
+            (cast<ConstantInt>(II->getOperand(2))->getSExtValue()) / 4);
       else if (isa<Instruction>(II->getArgOperand(2))) {
         // When it is not a const value and it is not a function argument, we
         // create Row after the definition of II->getOperand(2) instead of