commit 4240c5112aedb174a3e1c1f071ea54e82168b130
author Mathieu Chartier <mathieuc@google.com> Tue May 27 10:10:11 2014 -0700
committer Mathieu Chartier <mathieuc@google.com> Wed May 28 12:22:23 2014 -0700
tree c858f3976cf0628e1cc3e0be1d0f4ad1aa578a2a
parent 1b1aea22c0cc5567b5967590cb3f949cc45c3a9a

Don't create bump pointer spaces unless necessary.

We now only create these spaces if the foreground or background
collector is a moving collector.

Bug: 13564922
Bug: 14059466
Change-Id: I9f83c7881bc28423b78d5249c8e8854add91b60c

runtime/gc/collector/semi_space.cc

diff --git a/runtime/gc/collector/semi_space.cc b/runtime/gc/collector/semi_space.cc
index d4e26ab..e5bb1cc 100644
--- a/runtime/gc/collector/semi_space.cc
+++ b/runtime/gc/collector/semi_space.cc
@@ -272,12 +272,12 @@
   from_space_->Clear();
   VLOG(heap) << "Protecting from_space_: " << *from_space_;
   from_space_->GetMemMap()->Protect(kProtectFromSpace ? PROT_NONE : PROT_READ);
-  if (swap_semi_spaces_) {
-    heap_->SwapSemiSpaces();
-  }
   timings_.StartSplit("PreSweepingGcVerification");
   heap_->PreSweepingGcVerification(this);
   timings_.EndSplit();
+  if (swap_semi_spaces_) {
+    heap_->SwapSemiSpaces();
+  }
 }
 
 void SemiSpace::UpdateAndMarkModUnion() {