[Lldb-commits] [lldb] r176992 - <rdar://problem/13415471>

[Han Ming Ong]

Author: hanming
Date: Wed Mar 13 17:51:04 2013
New Revision: 176992

URL: http://llvm.org/viewvc/llvm-project?rev=176992&view=rev
Log:
<rdar://problem/13415471>

Don't get dirty page size if we are not going to send it back

Modified:
    lldb/trunk/tools/debugserver/source/MacOSX/MachVMMemory.cpp

Modified: lldb/trunk/tools/debugserver/source/MacOSX/MachVMMemory.cpp
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/tools/debugserver/source/MacOSX/MachVMMemory.cpp?rev=176992&r1=176991&r2=176992&view=diff
==============================================================================
--- lldb/trunk/tools/debugserver/source/MacOSX/MachVMMemory.cpp (original)
+++ lldb/trunk/tools/debugserver/source/MacOSX/MachVMMemory.cpp Wed Mar 13 17:51:04 2013
@@ -413,12 +413,12 @@ MachVMMemory::GetMemoryProfile(DNBProfil
     
         rsize = ti.resident_size;
         vsize = ti.virtual_size;
-    }
-    
-    if (scanType & eProfileMemoryDirtyPage)
-    {
-        // This uses vmmap strategy. We don't use the returned rsize for now. We prefer to match top's version since that's what we do for the rest of the metrics.
-        GetRegionSizes(task, rsize, dirty_size);
+        
+        if (scanType & eProfileMemoryDirtyPage)
+        {
+            // This uses vmmap strategy. We don't use the returned rsize for now. We prefer to match top's version since that's what we do for the rest of the metrics.
+            GetRegionSizes(task, rsize, dirty_size);
+        }
     }
     
     return true;