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

Issue 14845018: Allow for no samples in test-cpu-profiler/SampleWhenFrameIsNotSetup (Closed)

Created:
7 years, 7 months ago by yurys
Modified:
7 years, 7 months ago
CC:
v8-dev
Visibility:
Public.

Description

Allow for no samples in test-cpu-profiler/SampleWhenFrameIsNotSetup The test should only check that there are no sample stacks that never possible in the JS code being profiled. BUG=v8:2628 R=svenpanne@chromium.org Committed: https://code.google.com/p/v8/source/detail?r=14723

Patch Set 1 #

Patch Set 2 : Rebase #

Unified diffs Side-by-side diffs Delta from patch set Stats (+5 lines, -6 lines) Patch
M test/cctest/cctest.status View 2 chunks +2 lines, -5 lines 0 comments Download
M test/cctest/test-cpu-profiler.cc View 1 chunk +3 lines, -1 line 0 comments Download

Messages

Total messages: 6 (0 generated)
yurys
7 years, 7 months ago (2013-05-16 21:18:11 UTC) #1
Sven Panne
lgtm
7 years, 7 months ago (2013-05-17 06:53:57 UTC) #2
yurys
Committed patchset #2 manually as r14723 (presubmit successful).
7 years, 7 months ago (2013-05-17 15:10:33 UTC) #3
Sven Panne
Hmmm, somehow SampleWhenFrameIsNotSetup is still flaky, CheckChildrenNames is unconditionally called and sometimes has a failing ...
7 years, 7 months ago (2013-05-21 10:38:29 UTC) #4
yurys
On 2013/05/21 10:38:29, Sven Panne wrote: > Hmmm, somehow SampleWhenFrameIsNotSetup is still flaky, CheckChildrenNames is ...
7 years, 7 months ago (2013-05-23 05:08:47 UTC) #5
yurys
7 years, 7 months ago (2013-05-23 05:24:22 UTC) #6
Message was sent while issue was closed.
On 2013/05/23 05:08:47, Yury Semikhatsky wrote:
> On 2013/05/21 10:38:29, Sven Panne wrote:
> > Hmmm, somehow SampleWhenFrameIsNotSetup is still flaky, CheckChildrenNames
is
> > unconditionally called and sometimes has a failing CHECK. A new fix would be
> > highly appreciated, otherwise we'll have to disable the test again.
> 
> This is weird. Do you remember what was the unexpected node under root? I
didn't
> see this locally.

Ok, I see it failed on windows but not sure why, the profile dumped looks fine
and should have passed all the checks in the test, will investigate:

=== cctest/test-cpu-profiler/SampleWhenFrameIsNotSetup ===
--- stdout ---
[Top down]:
    2     0   (root) [-1] #1
    2     2    (program) [-1] #2
Command: E:\b\build\slave\v8-win32\build\v8\build\Release\cctest.exe
test-cpu-profiler/SampleWhenFrameIsNotSetup --nobreak-on-abort
--nodead-code-elimination --nofold-constants
--testing_serialization_file=E:\b\build\slave\v8-win32\build\v8\out\.serdes\serdes_test-cpu-profiler/SampleWhenFrameIsNotSetup

Powered by Google App Engine
This is Rietveld 408576698