diff options
author | Douglas Gregor <dgregor@apple.com> | 2008-12-11 20:41:00 +0000 |
---|---|---|
committer | Douglas Gregor <dgregor@apple.com> | 2008-12-11 20:41:00 +0000 |
commit | e267ff35b2f4e9d2b0d8bf24109d41cc7398b61b (patch) | |
tree | 568838468f95906278ae3e3b1753e3ad21e58041 /lib/Analysis/RegionStore.cpp | |
parent | 7415326dcbba6f333f5f7708ac1bf4982f3adb5a (diff) |
Address some comments on the name lookup/DeclContext patch from Chris
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@60897 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Analysis/RegionStore.cpp')
-rw-r--r-- | lib/Analysis/RegionStore.cpp | 11 |
1 files changed, 5 insertions, 6 deletions
diff --git a/lib/Analysis/RegionStore.cpp b/lib/Analysis/RegionStore.cpp index f8fc88612e..e065c5c410 100644 --- a/lib/Analysis/RegionStore.cpp +++ b/lib/Analysis/RegionStore.cpp @@ -433,13 +433,12 @@ SVal RegionStoreManager::RetrieveStruct(Store store, const TypedRegion* R) { llvm::ImmutableList<SVal> StructVal = getBasicVals().getEmptySValList(); - for (DeclContext::reverse_decl_iterator Mem = RD->decls_rbegin(); - Mem != RD->decls_rend(); ++Mem) { - FieldDecl *FD = dyn_cast<FieldDecl>(*Mem); - if (!FD) - continue; + std::vector<FieldDecl *> Fields(RD->field_begin(), RD->field_end()); - FieldRegion* FR = MRMgr.getFieldRegion(FD, R); + for (std::vector<FieldDecl *>::reverse_iterator Field = Fields.rbegin(), + FieldEnd = Fields.rend(); + Field != FieldEnd; ++Field) { + FieldRegion* FR = MRMgr.getFieldRegion(*Field, R); RegionBindingsTy B(static_cast<const RegionBindingsTy::TreeTy*>(store)); RegionBindingsTy::data_type* data = B.lookup(FR); |