[llvm] r373467 - MemorySSAUpdater::applyInsertUpdates - silence static analyzer dyn_cast<MemoryAccess> null dereference warning. NFCI.
Simon Pilgrim via llvm-commits
llvm-commits at lists.llvm.org
Wed Oct 2 06:09:12 PDT 2019
Author: rksimon
Date: Wed Oct 2 06:09:12 2019
New Revision: 373467
URL: http://llvm.org/viewvc/llvm-project?rev=373467&view=rev
Log:
MemorySSAUpdater::applyInsertUpdates - silence static analyzer dyn_cast<MemoryAccess> null dereference warning. NFCI.
The static analyzer is warning about a potential null dereference, but we should be able to use cast<MemoryAccess> directly and if not assert will fire for us.
Modified:
llvm/trunk/lib/Analysis/MemorySSAUpdater.cpp
Modified: llvm/trunk/lib/Analysis/MemorySSAUpdater.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Analysis/MemorySSAUpdater.cpp?rev=373467&r1=373466&r2=373467&view=diff
==============================================================================
--- llvm/trunk/lib/Analysis/MemorySSAUpdater.cpp (original)
+++ llvm/trunk/lib/Analysis/MemorySSAUpdater.cpp Wed Oct 2 06:09:12 2019
@@ -1082,7 +1082,7 @@ void MemorySSAUpdater::applyInsertUpdate
for (; UI != E;) {
Use &U = *UI;
++UI;
- MemoryAccess *Usr = dyn_cast<MemoryAccess>(U.getUser());
+ MemoryAccess *Usr = cast<MemoryAccess>(U.getUser());
if (MemoryPhi *UsrPhi = dyn_cast<MemoryPhi>(Usr)) {
BasicBlock *DominatedBlock = UsrPhi->getIncomingBlock(U);
if (!DT.dominates(DominatingBlock, DominatedBlock))
More information about the llvm-commits
mailing list