https://codereview.chromium.org/14253015/diff/2001/src/cpu-profiler.cc
File src/cpu-profiler.cc (right):
https://codereview.chromium.org/14253015/diff/2001/src/cpu-profiler.cc#newcode510
src/cpu-profiler.cc:510: printf("CompilationInfo size = %lu\n",
sizeof(CompilationInfo));
On 2013/04/26 13:36:56, loislo wrote:
please remove that
Done.
https://codereview.chromium.org/14253015/diff/2001/src/cpu-profiler.cc#newcode510
src/cpu-profiler.cc:510: printf("CompilationInfo size = %lu\n",
sizeof(CompilationInfo));
On 2013/04/26 13:36:56, loislo wrote:
please remove that
Done.
https://codereview.chromium.org/14253015/diff/2001/src/profile-generator.cc
File src/profile-generator.cc (right):
https://codereview.chromium.org/14253015/diff/2001/src/profile-generator.cc#newcode915
src/profile-generator.cc:915: return;
On 2013/04/26 13:36:56, loislo wrote:
looks like you are skipping the other stack traces.
Yes, this is intentional, the comment above describes that.
https://codereview.chromium.org/14253015/diff/2001/src/profile-generator.cc#newcode919
src/profile-generator.cc:919: return;
On 2013/04/26 13:36:56, loislo wrote:
ditto
See above.
https://codereview.chromium.org/14253015/diff/2001/src/profile-generator.cc#newcode921
src/profile-generator.cc:921: *entry++ = pc_entry;
On 2013/04/26 13:36:56, loislo wrote:
sounds like you are adding zero entry if the mapping wasn't found.
This part didn't change, it is normal case - pc could have pointed into
some stub code e.g.
https://codereview.chromium.org/14253015/
--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/groups/opt_out.
