- 12 Feb, 2015 26 commits
-
-
mstarzinger authored
R=titzer@chromium.org TEST=cctest/test-js-typed-lowering Review URL: https://codereview.chromium.org/925583002 Cr-Commit-Position: refs/heads/master@{#26626}
-
yangguo authored
This is to ensure that immutable immortal objects created during bootstrapping are not relocated. R=hpayer@chromium.org Review URL: https://codereview.chromium.org/905773004 Cr-Commit-Position: refs/heads/master@{#26625}
-
ishell authored
Review URL: https://codereview.chromium.org/917743004 Cr-Commit-Position: refs/heads/master@{#26624}
-
rossberg authored
Fixes the TSAN issue. R=mstarzinger@chromium.org NOTRY=true BUG= Review URL: https://codereview.chromium.org/923573002 Cr-Commit-Position: refs/heads/master@{#26623}
-
ulan authored
NOTREECHECKS=true BUG= TBR=verwaest@chromium.org Review URL: https://codereview.chromium.org/917193002 Cr-Commit-Position: refs/heads/master@{#26622}
-
jarin authored
R=rossberg@chromium.org BUG= Review URL: https://codereview.chromium.org/904863002 Cr-Commit-Position: refs/heads/master@{#26621}
-
marja authored
(To minimize the diff of actually interesting unifications.) R=rossberg@chromium.org BUG= Review URL: https://codereview.chromium.org/915383002 Cr-Commit-Position: refs/heads/master@{#26620}
-
rossberg authored
R=marja@chromium.org BUG= Review URL: https://codereview.chromium.org/917703003 Cr-Commit-Position: refs/heads/master@{#26619}
-
chunyang.dai authored
To solence C++ compiler warnings because is_mutable_ is only for CHECK. BUG= Review URL: https://codereview.chromium.org/922673002 Cr-Commit-Position: refs/heads/master@{#26618}
-
jochen authored
Also make sure that the function body really produces only one function literal. LOG=y BUG=none R=yangguo@chromium.org Review URL: https://codereview.chromium.org/925433002 Cr-Commit-Position: refs/heads/master@{#26617}
-
loislo authored
Revert of CPUProfiler: Push deopt reason further to ProfileNode. (patchset #1 id:1 of https://codereview.chromium.org/919953002/) Reason for revert: static initializers broke the build Original issue's description: > CPUProfiler: Push deopt reason further to ProfileNode. > > 1) create beefy RelocInfo table when cpu profiler is active, so if a function > was optimized when profiler was active RelocInfo would get separate DeoptInfo > for the each deopt case. > > 2) push DeoptInfo from CodeEntry to ProfileNode. > When deopt happens we put the info collected on #1 into CodeEntry and record stack sample. > On the sampling thread we grab the deopt data and append it to the corresponding ProfileNode deopts list. > > Sample profile dump. > [Top down]: > 0 (root) 0 #1 > 1 29 #2 > 5 test 29 #3 > 3 opt_function 29 #4 > deopted at 52 with reason 'not a heap number' > deopted at 71 with reason 'division by zero' > > BUG=452067 > LOG=n > > Committed: https://crrev.com/ce8701b247d3c6604f24f17a90c02d17b4417f54 > Cr-Commit-Position: refs/heads/master@{#26615} TBR=jarin@chromium.org,svenpanne@chromium.org,yurys@chromium.org,alph@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=452067 Review URL: https://codereview.chromium.org/915173005 Cr-Commit-Position: refs/heads/master@{#26616}
-
loislo authored
1) create beefy RelocInfo table when cpu profiler is active, so if a function was optimized when profiler was active RelocInfo would get separate DeoptInfo for the each deopt case. 2) push DeoptInfo from CodeEntry to ProfileNode. When deopt happens we put the info collected on #1 into CodeEntry and record stack sample. On the sampling thread we grab the deopt data and append it to the corresponding ProfileNode deopts list. Sample profile dump. [Top down]: 0 (root) 0 #1 1 29 #2 5 test 29 #3 3 opt_function 29 #4 deopted at 52 with reason 'not a heap number' deopted at 71 with reason 'division by zero' BUG=452067 LOG=n Review URL: https://codereview.chromium.org/919953002 Cr-Commit-Position: refs/heads/master@{#26615}
-
ulan authored
BUG= Review URL: https://codereview.chromium.org/871253005 Cr-Commit-Position: refs/heads/master@{#26614}
-
mstarzinger authored
R=jarin@chromium.org Review URL: https://codereview.chromium.org/918143002 Cr-Commit-Position: refs/heads/master@{#26613}
-
marja authored
Parser must be able to operate independent of Isolate and the V8 heap during parsing. After the heap-independent phase, there is a heap dependent phase, during which we internalize strings, handle errors, etc. This makes Isolate (also via CompilationInfo) unaccessible during parsing, and thus decreases the probability of accidental code changes which would add heap-dependent operations into the heap-independent phase. Since Isolate is also accessible via CompilationInfo, now CompilationInfo is only passed to the entry points of parsing, and not stored in Parser. R=rossberg@chromium.org BUG= Review URL: https://codereview.chromium.org/908173003 Cr-Commit-Position: refs/heads/master@{#26612}
-
bmeurer authored
Google style guide forbids -inl.h headers. R=svenpanne@chromium.org Review URL: https://codereview.chromium.org/922633002 Cr-Commit-Position: refs/heads/master@{#26611}
-
svenpanne authored
Use a fake code stub instead, basically following the null object pattern. Review URL: https://codereview.chromium.org/918973002 Cr-Commit-Position: refs/heads/master@{#26610}
-
bmeurer authored
Review URL: https://codereview.chromium.org/924453002 Cr-Commit-Position: refs/heads/master@{#26609}
-
danno authored
Previously, emitting two more more unique source positions at the same pc would generate two or more RelocInfo entries. Now, only the last emitted source position for any pc is added to the RelocInfo. Review URL: https://codereview.chromium.org/908443002 Cr-Commit-Position: refs/heads/master@{#26608}
-
hpayer authored
NOTREECHECKS=true TBR=ulan@chromium.org Review URL: https://codereview.chromium.org/918113002 Cr-Commit-Position: refs/heads/master@{#26607}
-
Ben L. Titzer authored
R=mstarzinger@chromium.org BUG= Review URL: https://codereview.chromium.org/913183005 Cr-Commit-Position: refs/heads/master@{#26606}
-
hpayer authored
Just visit young array buffers during scavenge. Additionally keep the views in new space in a separate global list and move them to the corresponding array buffers when they get promoted. This reduces young generation garbage collections when many array buffers are allocated. BUG= Review URL: https://codereview.chromium.org/904633003 Cr-Commit-Position: refs/heads/master@{#26605}
-
mstarzinger authored
R=danno@chromium.org TEST=cctest/test-run-intrinsics/CallFunction Review URL: https://codereview.chromium.org/916873002 Cr-Commit-Position: refs/heads/master@{#26604}
-
cdai2 authored
port fdcf3e59 (r26599) original commit message: new classes: implement correct check for uninitialized this in 'super()' BUG= R=weiliang.lin@intel.com Review URL: https://codereview.chromium.org/913383002 Cr-Commit-Position: refs/heads/master@{#26603}
-
bmeurer authored
Rename tools/vim/v8.ycm_extra_conf.py to .ycm_extra_conf.py, so every YCM plugin (no matter if it's VIM or Emacs) should pick it up automatically. Also move the common tools/vim/ninja_output.py to tools/ninja/ as it's also not related to VIM. Review URL: https://codereview.chromium.org/904143003 Cr-Commit-Position: refs/heads/master@{#26602}
-
cdai2 authored
port 9b158fa7 (r26594) original commit message: new classes: implement default constructors BUG= R=weiliang.lin@intel.com Review URL: https://codereview.chromium.org/914403004 Cr-Commit-Position: refs/heads/master@{#26601}
-
- 11 Feb, 2015 14 commits
-
-
dcarney authored
TBR=machenbach@chromium.org BUG= Review URL: https://codereview.chromium.org/915113004 Cr-Commit-Position: refs/heads/master@{#26600}
-
dslomov authored
R=arv@chromium.org BUG=v8:3834 LOG=N Review URL: https://codereview.chromium.org/918603004 Cr-Commit-Position: refs/heads/master@{#26599}
-
akos.palfi authored
Port 9b158fa7 BUG= Review URL: https://codereview.chromium.org/914303003 Cr-Commit-Position: refs/heads/master@{#26598}
-
akos.palfi authored
Port bf49be39 BUG= Review URL: https://codereview.chromium.org/906813003 Cr-Commit-Position: refs/heads/master@{#26597}
-
chunyang.dai authored
port bf49be39 (r26572) original commit message: new classes: implement new.target passing to superclass constructor BUG= Review URL: https://codereview.chromium.org/912233003 Cr-Commit-Position: refs/heads/master@{#26596}
-
chunyang.dai authored
This error is exposed by this CL: https://codereview.chromium.org/817293005. This CL set all Buildin JS function to optimization_disabled. And set the bailout reason to "kBuiltinFunctionCannotBeOptimized". But in Runtime_SetCode() function. It copied the the compiler_hints which include the "bool optimization_disabled" value, But the corresponding Bailout reason is not copied. This leads to the check error in "CompilationInfo::AbortOptimization(...)" function. This issue is exposed on turbofan unsupported X87 platform. Crankshaft is invoked to compile the typed array function Uint8Array. BUG= Review URL: https://codereview.chromium.org/917643002 Cr-Commit-Position: refs/heads/master@{#26595}
-
dslomov authored
R=arv@chromium.org,rossberg@chromium.org BUG=v8:3834 LOG=N Review URL: https://codereview.chromium.org/917753002 Cr-Commit-Position: refs/heads/master@{#26594}
-
loislo authored
The offensive cl is https://codereview.chromium.org/874323003/ Test: ./out/arm64.debug/d8 --test --random-seed=-235865360 --turbo-deoptimization --turbo-filter=* --always-opt --debug-code --verify-heap --gc-interval=500 --stress-compaction test/mjsunit/mjsunit.js test/mjsunit/regress/regress-builtinbust-7.js --trace-deopt BUG=452067 TBR=dcarney, svenpanne LOG=n Review URL: https://codereview.chromium.org/917823002 Cr-Commit-Position: refs/heads/master@{#26593}
-
rodolph.perfetta authored
BUG= Review URL: https://codereview.chromium.org/905383003 Cr-Commit-Position: refs/heads/master@{#26592}
-
mstarzinger authored
R=jarin@chromium.org TEST=unittests/ControlEquivalenceTest Review URL: https://codereview.chromium.org/916823002 Cr-Commit-Position: refs/heads/master@{#26591}
-
cdai2 authored
port 5d68529b (r26546) original commit message: Parsing: Make Scope not know about Isolate. Scope, like Parser, must be able to operate independent of Isolate and the V8 heap (for background parsing). After the heap-independent phase, there is a heap dependent phase, during which we do operations such as scope anaylysis. This CL makes the phases explicit by not telling Scope about the Isolate too early (during the heap-independent phase, Scope should know nothing about Isolate). This decreases the probability of accidental code changes which would add heap-dependent operations into the heap-independent phase. BUG= R=weiliang.lin@intel.com Review URL: https://codereview.chromium.org/913163003 Cr-Commit-Position: refs/heads/master@{#26590}
-
titzer authored
R=mstarzinger@chromium.org BUG= Review URL: https://codereview.chromium.org/917583004 Cr-Commit-Position: refs/heads/master@{#26589}
-
cdai2 authored
port 86cae163 (r26545) original commit message: 1) Deoptimizer::Reason was replaced with Deoptimizer::DeoptInfo because it also has raw position. Also the old name clashes with DeoptReason enum. 2) c_entry_fp assignment call was added to EntryGenerator::Generate So we can calculate sp and have a chance to record the stack for the deopting function. btw it makes the test stable. 3) new kind of CodeEvents was added to cpu-profiler 4) GetDeoptInfo method was extracted from PrintDeoptLocation. So it could be reused in cpu profiler. BUG= R=weiliang.lin@intel.com Review URL: https://codereview.chromium.org/912403003 Cr-Commit-Position: refs/heads/master@{#26588}
-
adamk authored
This comment used to have a (commented-out) ASSERT attached to it. But that ASSERT was removed years ago, so the comment is now a non-sequitar. Review URL: https://codereview.chromium.org/918453002 Cr-Commit-Position: refs/heads/master@{#26587}
-