summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorGeorge Burgess IV <george.burgess.iv@gmail.com>2015-12-10 19:25:21 +0000
committerGeorge Burgess IV <george.burgess.iv@gmail.com>2015-12-10 19:25:21 +0000
commit0fc4e8b4e71bab14cfbcb256ebf71cef45b8ef2d (patch)
treeabf1ceae45a74c1c0b49bab6f8ae8ae69a779b49
parentc63c0d1cc091a17ae734022edd46566b8bd525b5 (diff)
downloadbcm5719-llvm-0fc4e8b4e71bab14cfbcb256ebf71cef45b8ef2d.tar.gz
bcm5719-llvm-0fc4e8b4e71bab14cfbcb256ebf71cef45b8ef2d.zip
[Sema] Replace pointer-to-map with a map. NFC.
llvm-svn: 255288
-rw-r--r--clang/lib/Sema/AnalysisBasedWarnings.cpp18
1 files changed, 6 insertions, 12 deletions
diff --git a/clang/lib/Sema/AnalysisBasedWarnings.cpp b/clang/lib/Sema/AnalysisBasedWarnings.cpp
index 549ab4f8447..5f74343fbd9 100644
--- a/clang/lib/Sema/AnalysisBasedWarnings.cpp
+++ b/clang/lib/Sema/AnalysisBasedWarnings.cpp
@@ -1339,20 +1339,16 @@ class UninitValsDiagReporter : public UninitVariablesHandler {
// the same as insertion order. This is needed to obtain a deterministic
// order of diagnostics when calling flushDiagnostics().
typedef llvm::MapVector<const VarDecl *, MappedType> UsesMap;
- UsesMap *uses;
+ UsesMap uses;
public:
- UninitValsDiagReporter(Sema &S) : S(S), uses(nullptr) {}
+ UninitValsDiagReporter(Sema &S) : S(S) {}
~UninitValsDiagReporter() override { flushDiagnostics(); }
MappedType &getUses(const VarDecl *vd) {
- if (!uses)
- uses = new UsesMap();
-
- MappedType &V = (*uses)[vd];
+ MappedType &V = uses[vd];
if (!V.getPointer())
V.setPointer(new UsesVec());
-
return V;
}
@@ -1366,10 +1362,7 @@ public:
}
void flushDiagnostics() {
- if (!uses)
- return;
-
- for (const auto &P : *uses) {
+ for (const auto &P : uses) {
const VarDecl *vd = P.first;
const MappedType &V = P.second;
@@ -1410,7 +1403,8 @@ public:
// Release the uses vector.
delete vec;
}
- delete uses;
+
+ uses.clear();
}
private:
OpenPOWER on IntegriCloud