Bug Summary

File:lib/CodeGen/MachineLoopInfo.cpp
Warning:line 131, column 31
Called C++ object pointer is null

Annotated Source Code

Press '?' to see keyboard shortcuts

clang -cc1 -triple x86_64-pc-linux-gnu -analyze -disable-free -disable-llvm-verifier -discard-value-names -main-file-name MachineLoopInfo.cpp -analyzer-store=region -analyzer-opt-analyze-nested-blocks -analyzer-checker=core -analyzer-checker=apiModeling -analyzer-checker=unix -analyzer-checker=deadcode -analyzer-checker=cplusplus -analyzer-checker=security.insecureAPI.UncheckedReturn -analyzer-checker=security.insecureAPI.getpw -analyzer-checker=security.insecureAPI.gets -analyzer-checker=security.insecureAPI.mktemp -analyzer-checker=security.insecureAPI.mkstemp -analyzer-checker=security.insecureAPI.vfork -analyzer-checker=nullability.NullPassedToNonnull -analyzer-checker=nullability.NullReturnedFromNonnull -analyzer-output plist -w -analyzer-config-compatibility-mode=true -mrelocation-model pic -pic-level 2 -mthread-model posix -mframe-pointer=none -fmath-errno -masm-verbose -mconstructor-aliases -munwind-tables -fuse-init-array -target-cpu x86-64 -dwarf-column-info -debugger-tuning=gdb -ffunction-sections -fdata-sections -resource-dir /usr/lib/llvm-10/lib/clang/10.0.0 -D _DEBUG -D _GNU_SOURCE -D __STDC_CONSTANT_MACROS -D __STDC_FORMAT_MACROS -D __STDC_LIMIT_MACROS -I /build/llvm-toolchain-snapshot-10~svn374877/build-llvm/lib/CodeGen -I /build/llvm-toolchain-snapshot-10~svn374877/lib/CodeGen -I /build/llvm-toolchain-snapshot-10~svn374877/build-llvm/include -I /build/llvm-toolchain-snapshot-10~svn374877/include -U NDEBUG -internal-isystem /usr/lib/gcc/x86_64-linux-gnu/6.3.0/../../../../include/c++/6.3.0 -internal-isystem /usr/lib/gcc/x86_64-linux-gnu/6.3.0/../../../../include/x86_64-linux-gnu/c++/6.3.0 -internal-isystem /usr/lib/gcc/x86_64-linux-gnu/6.3.0/../../../../include/x86_64-linux-gnu/c++/6.3.0 -internal-isystem /usr/lib/gcc/x86_64-linux-gnu/6.3.0/../../../../include/c++/6.3.0/backward -internal-isystem /usr/local/include -internal-isystem /usr/lib/llvm-10/lib/clang/10.0.0/include -internal-externc-isystem /usr/include/x86_64-linux-gnu -internal-externc-isystem /include -internal-externc-isystem /usr/include -O2 -Wno-unused-parameter -Wwrite-strings -Wno-missing-field-initializers -Wno-long-long -Wno-maybe-uninitialized -Wno-comment -std=c++14 -fdeprecated-macro -fdebug-compilation-dir /build/llvm-toolchain-snapshot-10~svn374877/build-llvm/lib/CodeGen -fdebug-prefix-map=/build/llvm-toolchain-snapshot-10~svn374877=. -ferror-limit 19 -fmessage-length 0 -fvisibility-inlines-hidden -stack-protector 2 -fgnuc-version=4.2.1 -fobjc-runtime=gcc -fdiagnostics-show-option -vectorize-loops -vectorize-slp -analyzer-output=html -analyzer-config stable-report-filename=true -faddrsig -o /tmp/scan-build-2019-10-15-233810-7101-1 -x c++ /build/llvm-toolchain-snapshot-10~svn374877/lib/CodeGen/MachineLoopInfo.cpp
1//===- MachineLoopInfo.cpp - Natural Loop Calculator ----------------------===//
2//
3// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
4// See https://llvm.org/LICENSE.txt for license information.
5// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
6//
7//===----------------------------------------------------------------------===//
8//
9// This file defines the MachineLoopInfo class that is used to identify natural
10// loops and determine the loop depth of various nodes of the CFG. Note that
11// the loops identified may actually be several natural loops that share the
12// same header node... not just a single natural loop.
13//
14//===----------------------------------------------------------------------===//
15
16#include "llvm/CodeGen/MachineLoopInfo.h"
17#include "llvm/Analysis/LoopInfoImpl.h"
18#include "llvm/CodeGen/MachineDominators.h"
19#include "llvm/CodeGen/Passes.h"
20#include "llvm/Config/llvm-config.h"
21#include "llvm/Support/Debug.h"
22#include "llvm/Support/raw_ostream.h"
23using namespace llvm;
24
25// Explicitly instantiate methods in LoopInfoImpl.h for MI-level Loops.
26template class llvm::LoopBase<MachineBasicBlock, MachineLoop>;
27template class llvm::LoopInfoBase<MachineBasicBlock, MachineLoop>;
28
29char MachineLoopInfo::ID = 0;
30INITIALIZE_PASS_BEGIN(MachineLoopInfo, "machine-loops",static void *initializeMachineLoopInfoPassOnce(PassRegistry &
Registry) {
31 "Machine Natural Loop Construction", true, true)static void *initializeMachineLoopInfoPassOnce(PassRegistry &
Registry) {
32INITIALIZE_PASS_DEPENDENCY(MachineDominatorTree)initializeMachineDominatorTreePass(Registry);
33INITIALIZE_PASS_END(MachineLoopInfo, "machine-loops",PassInfo *PI = new PassInfo( "Machine Natural Loop Construction"
, "machine-loops", &MachineLoopInfo::ID, PassInfo::NormalCtor_t
(callDefaultCtor<MachineLoopInfo>), true, true); Registry
.registerPass(*PI, true); return PI; } static llvm::once_flag
InitializeMachineLoopInfoPassFlag; void llvm::initializeMachineLoopInfoPass
(PassRegistry &Registry) { llvm::call_once(InitializeMachineLoopInfoPassFlag
, initializeMachineLoopInfoPassOnce, std::ref(Registry)); }
34 "Machine Natural Loop Construction", true, true)PassInfo *PI = new PassInfo( "Machine Natural Loop Construction"
, "machine-loops", &MachineLoopInfo::ID, PassInfo::NormalCtor_t
(callDefaultCtor<MachineLoopInfo>), true, true); Registry
.registerPass(*PI, true); return PI; } static llvm::once_flag
InitializeMachineLoopInfoPassFlag; void llvm::initializeMachineLoopInfoPass
(PassRegistry &Registry) { llvm::call_once(InitializeMachineLoopInfoPassFlag
, initializeMachineLoopInfoPassOnce, std::ref(Registry)); }
35
36char &llvm::MachineLoopInfoID = MachineLoopInfo::ID;
37
38bool MachineLoopInfo::runOnMachineFunction(MachineFunction &) {
39 releaseMemory();
40 LI.analyze(getAnalysis<MachineDominatorTree>().getBase());
41 return false;
42}
43
44void MachineLoopInfo::getAnalysisUsage(AnalysisUsage &AU) const {
45 AU.setPreservesAll();
46 AU.addRequired<MachineDominatorTree>();
47 MachineFunctionPass::getAnalysisUsage(AU);
48}
49
50MachineBasicBlock *MachineLoop::getTopBlock() {
51 MachineBasicBlock *TopMBB = getHeader();
52 MachineFunction::iterator Begin = TopMBB->getParent()->begin();
53 if (TopMBB->getIterator() != Begin) {
54 MachineBasicBlock *PriorMBB = &*std::prev(TopMBB->getIterator());
55 while (contains(PriorMBB)) {
56 TopMBB = PriorMBB;
57 if (TopMBB->getIterator() == Begin)
58 break;
59 PriorMBB = &*std::prev(TopMBB->getIterator());
60 }
61 }
62 return TopMBB;
63}
64
65MachineBasicBlock *MachineLoop::getBottomBlock() {
66 MachineBasicBlock *BotMBB = getHeader();
67 MachineFunction::iterator End = BotMBB->getParent()->end();
68 if (BotMBB->getIterator() != std::prev(End)) {
69 MachineBasicBlock *NextMBB = &*std::next(BotMBB->getIterator());
70 while (contains(NextMBB)) {
71 BotMBB = NextMBB;
72 if (BotMBB == &*std::next(BotMBB->getIterator()))
73 break;
74 NextMBB = &*std::next(BotMBB->getIterator());
75 }
76 }
77 return BotMBB;
78}
79
80MachineBasicBlock *MachineLoop::findLoopControlBlock() {
81 if (MachineBasicBlock *Latch = getLoopLatch()) {
82 if (isLoopExiting(Latch))
83 return Latch;
84 else
85 return getExitingBlock();
86 }
87 return nullptr;
88}
89
90DebugLoc MachineLoop::getStartLoc() const {
91 // Try the pre-header first.
92 if (MachineBasicBlock *PHeadMBB = getLoopPreheader())
93 if (const BasicBlock *PHeadBB = PHeadMBB->getBasicBlock())
94 if (DebugLoc DL = PHeadBB->getTerminator()->getDebugLoc())
95 return DL;
96
97 // If we have no pre-header or there are no instructions with debug
98 // info in it, try the header.
99 if (MachineBasicBlock *HeadMBB = getHeader())
100 if (const BasicBlock *HeadBB = HeadMBB->getBasicBlock())
101 return HeadBB->getTerminator()->getDebugLoc();
102
103 return DebugLoc();
104}
105
106MachineBasicBlock *
107MachineLoopInfo::findLoopPreheader(MachineLoop *L,
108 bool SpeculativePreheader) const {
109 if (MachineBasicBlock *PB = L->getLoopPreheader())
1
Assuming 'PB' is null
→
2
←
Taking false branch
→
110 return PB;
111
112 if (!SpeculativePreheader)
3
←
Assuming 'SpeculativePreheader' is true
→
4
←
Taking false branch
→
113 return nullptr;
114
115 MachineBasicBlock *HB = L->getHeader(), *LB = L->getLoopLatch();
116 if (HB->pred_size() != 2 || HB->hasAddressTaken())
5
←
Assuming the condition is false
→
6
←
Assuming the condition is false
→
7
←
Taking false branch
→
117 return nullptr;
118 // Find the predecessor of the header that is not the latch block.
119 MachineBasicBlock *Preheader = nullptr;
8
←
'Preheader' initialized to a null pointer value
→
120 for (MachineBasicBlock *P : HB->predecessors()) {
121 if (P == LB)
122 continue;
123 // Sanity.
124 if (Preheader)
125 return nullptr;
126 Preheader = P;
127 }
128
129 // Check if the preheader candidate is a successor of any other loop
130 // headers. We want to avoid having two loop setups in the same block.
131 for (MachineBasicBlock *S : Preheader->successors()) {
9
←
Called C++ object pointer is null
132 if (S == HB)
133 continue;
134 MachineLoop *T = getLoopFor(S);
135 if (T && T->getHeader() == S)
136 return nullptr;
137 }
138 return Preheader;
139}
140
141#if !defined(NDEBUG) || defined(LLVM_ENABLE_DUMP)
142LLVM_DUMP_METHOD__attribute__((noinline)) __attribute__((__used__)) void MachineLoop::dump() const {
143 print(dbgs());
144}
145#endif