Author: [email protected]
Date: Tue Jun 16 08:01:24 2009
New Revision: 2193

Modified:
    branches/bleeding_edge/tools/codemap.js

Log:
Use >>> instead of >> in order to cover the full 32 bit range when
handling addresses.
Review URL: http://codereview.chromium.org/125187

Modified: branches/bleeding_edge/tools/codemap.js
==============================================================================
--- branches/bleeding_edge/tools/codemap.js     (original)
+++ branches/bleeding_edge/tools/codemap.js     Tue Jun 16 08:01:24 2009
@@ -126,7 +126,7 @@
  devtools.profiler.CodeMap.prototype.markPages_ = function(start, end) {
    for (var addr = start; addr <= end;
         addr += devtools.profiler.CodeMap.PAGE_SIZE) {
-    this.pages_[addr >> devtools.profiler.CodeMap.PAGE_ALIGNMENT] = 1;
+    this.pages_[addr >>> devtools.profiler.CodeMap.PAGE_ALIGNMENT] = 1;
    }
  };

@@ -155,7 +155,7 @@
   * @param {number} addr Address.
   */
  devtools.profiler.CodeMap.prototype.findEntry = function(addr) {
-  var pageAddr = addr >> devtools.profiler.CodeMap.PAGE_ALIGNMENT;
+  var pageAddr = addr >>> devtools.profiler.CodeMap.PAGE_ALIGNMENT;
    if (pageAddr in this.pages_) {
      return this.findInTree_(this.statics_, addr);
    }

--~--~---------~--~----~------------~-------~--~----~
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
-~----------~----~----~----~------~----~------~--~---

Reply via email to