From 76a1be05fa00638532bfda02839f1847ab15ede7 Mon Sep 17 00:00:00 2001 From: Simon Pilgrim Date: Sun, 6 Jun 2021 15:05:32 +0100 Subject: [PATCH] AssumeBundleQueries.cpp - don't dereference a dyn_cast<> result. NFCI. Use cast<> instead which will assert that the cast is correct and not just return null - the match() should have already failed if the cast isn't valid anyhow. Fixes static analysis warning. --- llvm/lib/Analysis/AssumeBundleQueries.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/llvm/lib/Analysis/AssumeBundleQueries.cpp b/llvm/lib/Analysis/AssumeBundleQueries.cpp index 11f3e03..519d27a 100644 --- a/llvm/lib/Analysis/AssumeBundleQueries.cpp +++ b/llvm/lib/Analysis/AssumeBundleQueries.cpp @@ -130,10 +130,10 @@ bool llvm::isAssumeWithEmptyBundle(AssumeInst &Assume) { } static CallInst::BundleOpInfo *getBundleFromUse(const Use *U) { - auto *Intr = dyn_cast(U->getUser()); if (!match(U->getUser(), m_Intrinsic(m_Unless(m_Specific(U->get()))))) return nullptr; + auto *Intr = cast(U->getUser()); return &Intr->getBundleOpInfoForOperand(U->getOperandNo()); } -- 2.7.4