Command line jsc should report memory footprint in bytes
authormsaboff@apple.com <msaboff@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 3 Oct 2018 23:24:10 +0000 (23:24 +0000)
committermsaboff@apple.com <msaboff@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 3 Oct 2018 23:24:10 +0000 (23:24 +0000)
https://bugs.webkit.org/show_bug.cgi?id=190267

Reviewed by Mark Lam.

Change to leave the footprint values from the system unmodified.

* jsc.cpp:
(JSCMemoryFootprint::finishCreation):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@236814 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/JavaScriptCore/ChangeLog
Source/JavaScriptCore/jsc.cpp

index 6b3dee8..fe39196 100644 (file)
@@ -1,3 +1,15 @@
+2018-10-03  Michael Saboff  <msaboff@apple.com>
+
+        Command line jsc should report memory footprint in bytes
+        https://bugs.webkit.org/show_bug.cgi?id=190267
+
+        Reviewed by Mark Lam.
+
+        Change to leave the footprint values from the system unmodified.
+
+        * jsc.cpp:
+        (JSCMemoryFootprint::finishCreation):
+
 2018-10-03  Mark Lam  <mark.lam@apple.com>
 
         Suppress unreachable code warning for LLIntAssembly.h code.
index f6c1e82..63b29b6 100644 (file)
@@ -1231,9 +1231,8 @@ public:
 
         MemoryFootprint footprint = MemoryFootprint::now();
 
-        // Report sizes in KBytes so that values up to GB are still integers.
-        addProperty(vm, "current", jsNumber(footprint.current / 1024));
-        addProperty(vm, "peak", jsNumber(footprint.peak / 1024));
+        addProperty(vm, "current", jsNumber(footprint.current));
+        addProperty(vm, "peak", jsNumber(footprint.peak));
     }
 
     DECLARE_INFO;