From: David Blaikie <dblaikie@gmail.com>
Date: Wed, 20 Feb 2013 22:23:01 +0000 (+0000)
Subject: Use op-> directly rather than via Optional<T>::getPointer.
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=0b9c328bb47b38ef6ff877a42e8a90a31ab0e2e1;p=clang

Use op-> directly rather than via Optional<T>::getPointer.

Post-commit CR feedback from Jordan Rose regarding r175594.

git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@175677 91177308-0d34-0410-b5e6-96231b3b80d8
---

diff --git a/lib/StaticAnalyzer/Core/RegionStore.cpp b/lib/StaticAnalyzer/Core/RegionStore.cpp
index 39417afbd0..b5e31d2f52 100644
--- a/lib/StaticAnalyzer/Core/RegionStore.cpp
+++ b/lib/StaticAnalyzer/Core/RegionStore.cpp
@@ -1265,7 +1265,7 @@ RegionStoreManager::getLazyBinding(RegionBindingsConstRef B,
   if (originalRegion != R) {
     if (Optional<SVal> OV = B.getDefaultBinding(R)) {
       if (llvm::Optional<nonloc::LazyCompoundVal> V =
-              OV.getPointer()->getAs<nonloc::LazyCompoundVal>())
+              OV->getAs<nonloc::LazyCompoundVal>())
         return std::make_pair(V->getStore(), V->getRegion());
     }
   }
@@ -1671,7 +1671,7 @@ NonLoc RegionStoreManager::createLazyBinding(RegionBindingsConstRef B,
   // don't create a new lazy binding.
   if (Optional<SVal> V = B.getDefaultBinding(R)) {
     if (llvm::Optional<nonloc::LazyCompoundVal> LCV =
-            V.getPointer()->getAs<nonloc::LazyCompoundVal>()) {
+            V->getAs<nonloc::LazyCompoundVal>()) {
       QualType RegionTy = R->getValueType();
       QualType SourceRegionTy = LCV->getRegion()->getValueType();
       if (Ctx.hasSameUnqualifiedType(RegionTy, SourceRegionTy))