Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(425)

Issue 8491061: More info in disassembled code (Closed)

Created:
9 years, 1 month ago by hausner
Modified:
9 years, 1 month ago
Reviewers:
siva
CC:
reviews_dartlang.org
Visibility:
Public.

Description

More info in disassembled code Print out first 5 elements of an array object, rather than just 'Array'. This helps in particular in disassembled method and stub calls which now print the content of the IC cache and ArgumentsDescriptor arrays. E.g.: 0x7404a5 ff7508 push [ebp+0x8] 0x7404a8 ff750c push [ebp+0xc] 0x7404ab b9012f7000 mov ecx,0x702f01 Array[bitAndFromInteger, 1, null, null] 0x7404b0 bafd050003 mov edx,0x30005fd Array[2, 2, null] 0x7404b5 e8b6fcffff call 0x740170 [stub: InlineCache] 0x7404ba 83c408 add esp,0x8 Committed: https://code.google.com/p/dart/source/detail?r=1493

Patch Set 1 #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+17 lines, -1 line) Patch
M runtime/vm/disassembler_ia32.cc View 1 chunk +17 lines, -1 line 2 comments Download

Messages

Total messages: 4 (0 generated)
hausner
9 years, 1 month ago (2011-11-12 00:20:05 UTC) #1
hausner
TBR.
9 years, 1 month ago (2011-11-12 00:45:35 UTC) #2
siva
lgtm http://codereview.chromium.org/8491061/diff/1/runtime/vm/disassembler_ia32.cc File runtime/vm/disassembler_ia32.cc (right): http://codereview.chromium.org/8491061/diff/1/runtime/vm/disassembler_ia32.cc#newcode399 runtime/vm/disassembler_ia32.cc:399: const Array& arr = Array::CheckedHandle(obj.raw()); You have already ...
9 years, 1 month ago (2011-11-12 01:11:55 UTC) #3
hausner
9 years, 1 month ago (2011-11-12 01:21:08 UTC) #4
Thanks, will fix in a follow-up checkin.

Powered by Google App Engine
This is Rietveld 408576698