- 04 Feb, 2016 19 commits
-
-
mythria authored
Removes skips for two tests in cctest that are no longer crashing with ignition. BUG=v8:4680 LOG=N Review URL: https://codereview.chromium.org/1668843003 Cr-Commit-Position: refs/heads/master@{#33732}
-
machenbach authored
BUG= Review URL: https://codereview.chromium.org/1668833002 Cr-Commit-Position: refs/heads/master@{#33731}
-
rmcilroy authored
Moves the stack check from the function entry trampoline to instead be after function activation using an explicit StackCheck bytecode. Also add stack checks on back edges of loops. BUG=v8:4280,v8:4678 LOG=N Review URL: https://codereview.chromium.org/1665853002 Cr-Commit-Position: refs/heads/master@{#33730}
-
yangguo authored
This is to avoid polluting fuzzer seeds with the --ignition flag until we figure out something better. TBR=mstarzinger@chromium.org Review URL: https://codereview.chromium.org/1661333002 Cr-Commit-Position: refs/heads/master@{#33729}
-
sigurds authored
This is also a refactoring of the merge function in escape analysis. BUG=v8:4586 LOG=n Review URL: https://codereview.chromium.org/1654163003 Cr-Commit-Position: refs/heads/master@{#33728}
-
machenbach authored
Crashes after https://codereview.chromium.org/1667673003 BUG=v8:4280 LOG=N NOTRY=true TBR=rmcilroy@chromium.org Review URL: https://codereview.chromium.org/1669783002 Cr-Commit-Position: refs/heads/master@{#33727}
-
yangguo authored
This change adds the basic infrastructure to record source positions for bytecode. R=rmcilroy@chromium.org, vogelheim@chromium.org BUG=v8:4960 LOG=N Review URL: https://codereview.chromium.org/1662983002 Cr-Commit-Position: refs/heads/master@{#33726}
-
machenbach authored
BUG=v8:4280 LOG=N NOTRY=true TBR=rmcilroy@chromium.org Review URL: https://codereview.chromium.org/1661193003 Cr-Commit-Position: refs/heads/master@{#33725}
-
machenbach authored
BUG=v8:4280 LOG=N Review URL: https://codereview.chromium.org/1667673003 Cr-Commit-Position: refs/heads/master@{#33724}
-
titzer authored
This cleans up and simplifyies handling the bytes followin an opcode with little helper structs that will be useful in the interpreter and already have been in keeping OpcodeArity and OpcodeLength up to date with the decoder. R=bradnelson@chromium.org, ahaas@chromium.org BUG= Review URL: https://codereview.chromium.org/1664883002 Cr-Commit-Position: refs/heads/master@{#33723}
-
mythria authored
Adds implementation and tests for rest parameters to interpreter. BUG=v8:4280,v8:4683 LOG=N Review URL: https://codereview.chromium.org/1664593003 Cr-Commit-Position: refs/heads/master@{#33722}
-
ahaas authored
The goal of the Int64Reducer is to replace all int64 nodes in a tf graph with a set of int32 nodes such that 64 bit tf functions can be executed on 32 bit platforms. At the moment the Int64Reducer only replaces Int64Constants, TruncateInt64ToInt32, and Word64And. R=titzer@chromium.org Review URL: https://codereview.chromium.org/1655883002 Cr-Commit-Position: refs/heads/master@{#33721}
-
bmeurer authored
The parser should have never done this. The AstNumbering takes care of disabling compilers for certain language constructs. TBR=rossberg@chromium.org Review URL: https://codereview.chromium.org/1669523004 Cr-Commit-Position: refs/heads/master@{#33720}
-
hablich authored
With the combination of the WATCHLISTS feature and create_release.py it is possible to notify the mailing list v8-merges@googlegroups.com on pending merges. On master this notification is deactivated. R=machenbach@chromium.org NOTRY=true Review URL: https://codereview.chromium.org/1663003003 Cr-Commit-Position: refs/heads/master@{#33719}
-
mvstanton authored
BUG= Review URL: https://codereview.chromium.org/1647123002 Cr-Commit-Position: refs/heads/master@{#33718}
-
bmeurer authored
The spec requires all Math functions to first call ToNumber on all arguments before doing any other observable operation. So early return in case of Infinity is not valid. Drive-by-fix: Remove the use of %_Arguments / %_ArgumentsLength and use (strict) arguments instead of allocating a temporary InternalArray explicitly. R=yangguo@chromium.org Review URL: https://codereview.chromium.org/1669773002 Cr-Commit-Position: refs/heads/master@{#33717}
-
bmeurer authored
No need to micro-optimize here, and in almost all cases here, using arguments should result in roughly the same code w/ Crankshaft anyway. R=yangguo@chromium.org Review URL: https://codereview.chromium.org/1664513007 Cr-Commit-Position: refs/heads/master@{#33716}
-
mtrofin authored
So far, we've been moving down gaps wholesale. This change moves individual move operations instead. This improves some benchmarks, and should overall reduce code size, because it improves the chance of reducing the number of moves. For example, there are improvements on x64 in Emscripten (Bullet, in particular) , JetStream geomean, Embenchen (zlib). In the process of making this change, I noticed we can separate the tasks performed by the move optimizer, as follows: - group gaps into 1 - push gaps down, jumping instructions (these 2 were together before) - merge blocks (and then push gaps down) - finalize We can do without a finalization list. This avoids duplicating storage - we already have the list of instructions; it also simplifies the logic, since, with this change, we may process an instruction's gap twice. Compile time doesn't regress much (see pathological cases), but we may want to avoid the allocations of the few sets used in the new code. I'll do that in a subsequent change. BUG= Review URL: https://codereview.chromium.org/1634093002 Cr-Commit-Position: refs/heads/master@{#33715}
-
v8-autoroll authored
Rolling v8/buildtools to e27b1f1459452013ce59c9d5dbc93c88982cb76e Rolling v8/tools/clang to 6449b18afaa80290fd8930c3a42c80908505f41f TBR=machenbach@chromium.org,vogelheim@chromium.org,hablich@chromium.org Review URL: https://codereview.chromium.org/1660143007 Cr-Commit-Position: refs/heads/master@{#33714}
-
- 03 Feb, 2016 21 commits
-
-
aseemgarg authored
R=bradnelson@chromium.org BUG=https://bugs.chromium.org/p/v8/issues/detail?id=4203 TEST=asm-wasm.js LOG=N Review URL: https://codereview.chromium.org/1664993002 Cr-Commit-Position: refs/heads/master@{#33713}
-
adamk authored
This removes --harmony-completion, --harmony-concat-spreadable, and --harmony-tolength and moves the appropriate tests from harmony/ to es6/. Review URL: https://codereview.chromium.org/1667453002 Cr-Commit-Position: refs/heads/master@{#33712}
-
mbrandy authored
R=joransiu@ca.ibm.com, jyan@ca.ibm.com, michael_dawson@ca.ibm.com BUG= Review URL: https://codereview.chromium.org/1663133002 Cr-Commit-Position: refs/heads/master@{#33711}
-
mbrandy authored
Port 44ec23ac R=hpayer@chromium.org, joransiu@ca.ibm.com, jyan@ca.ibm.com, michael_dawson@ca.ibm.com BUG=chromium:561449 LOG=n Review URL: https://codereview.chromium.org/1669463002 Cr-Commit-Position: refs/heads/master@{#33710}
-
kozyatinskiy authored
A lot of deverlopers use deprecated sourceURL syntax. We should add console warning message before removing this. Original CL: https://codereview.chromium.org/1495633002/ BUG=chromium:558998 LOG=Y R=yangguo@chromium.org,hablich@chromium.org,adamk@chromium.org Review URL: https://codereview.chromium.org/1654223003 Cr-Commit-Position: refs/heads/master@{#33709}
-
mythria authored
Skips cctest/test-heap/NoWeakHashTableLeakWithIncrementalMarking that is crashing on arm64. BUG=v8:4280 LOG=N NOTRY=true Review URL: https://codereview.chromium.org/1660613005 Cr-Commit-Position: refs/heads/master@{#33708}
-
oth authored
Unifies the meaning of kRegCount8 and kRegCount16 across bytecodes. Call and CallJSRuntime had a slightly different use of the register count operand. From this change forth, register count operands are always based off of the previous register operand. BUG=v8:4280,v8:4675 LOG=N Review URL: https://codereview.chromium.org/1659023002 Cr-Commit-Position: refs/heads/master@{#33707}
-
yangguo authored
BUG=v8:4717 LOG=N Review URL: https://codereview.chromium.org/1663543003 Cr-Commit-Position: refs/heads/master@{#33706}
-
mythria authored
Adds implementation and tests for with statement to interprter. BUG=v8:4280,v8:4684 LOG=N Review URL: https://codereview.chromium.org/1656863002 Cr-Commit-Position: refs/heads/master@{#33705}
-
mstarzinger authored
This clears the currently pending message object whenever a try-block or a finally-block is being entered in interpreted code. The intention is to avoid memory leaks introduced by the message object. Also the message object is being restored when a finally-block exits. R=rmcilroy@chromium.org TEST=cctest/test-heap/MessageObjectLeak BUG=v8:4674 LOG=n Review URL: https://codereview.chromium.org/1651993002 Cr-Commit-Position: refs/heads/master@{#33704}
-
balazs.kilvady authored
BUG= Review URL: https://codereview.chromium.org/1628453002 Cr-Commit-Position: refs/heads/master@{#33703}
-
titzer authored
R=ahaas@chromium.org,jfb@chromium.org BUG= Review URL: https://codereview.chromium.org/1661713003 Cr-Commit-Position: refs/heads/master@{#33702}
-
verwaest authored
This was inconsistent in the spec in case of has vs get, set. Removing receiver==holder simplifies the lookup; so tentatively removing this additional check which was broken until yesterday anyway. See https://github.com/tc39/ecma262/issues/347 for more information. Review URL: https://codereview.chromium.org/1660903002 Cr-Commit-Position: refs/heads/master@{#33701}
-
bradnelson authored
Now that we have full fledged module instantiation, keeping alive a second interface seems clunky. Droping it and switching the tests. BUG= https://code.google.com/p/v8/issues/detail?id=4203 TEST=test-asm-validator,asm-wasm,other wasm R=titzer@chromium.org,aseemgarg@chromium.org LOG=N Review URL: https://codereview.chromium.org/1660083002 Cr-Commit-Position: refs/heads/master@{#33700}
-
jochen authored
BUG=none R=verwaest@chromium.org LOG=n Review URL: https://codereview.chromium.org/1653423002 Cr-Commit-Position: refs/heads/master@{#33699}
-
hablich authored
Revert of [api] Make ObjectTemplate::SetNativeDataProperty() work even if the ObjectTemplate does not have a … (patchset #3 id:80001 of https://codereview.chromium.org/1642223003/ ) Reason for revert: Fails a lot of layout tests and blocks the roll. Can be easily reproduced with a local Chromium checkout. Reference: https://codereview.chromium.org/1652413003/ Original issue's description: > [api] Make ObjectTemplate::SetNativeDataProperty() work even if the ObjectTemplate does not have a constructor. > > Previously ObjectTemplate::New() logic relied on the fact that all the accessor properties are already installed in the initial map of the function object of the constructor FunctionTemplate. > When the FunctionTemplate were instantiated the accessors of the instance templates from the whole inheritance chain were accumulated and added to the initial map. > ObjectTemplate::SetSetAccessor() used to explicitly ensure that the ObjectTemplate has a constructor and therefore an initial map to add all accessors to. > > The new approach is to add all the accessors and data properties to the object exactly when the ObjectTemplate is instantiated. In order to keep it fast we now cache the object boilerplates in the Isolate::template_instantiations_cache (the former function_cache), so the object creation turns to be a deep copying of the boilerplate object. > > This CL also prohibits non-primitive properties in ObjectTemplate to avoid potential cross-context leaks. > > BUG=chromium:579009 > LOG=Y > > Committed: https://crrev.com/6a118774244d087b5979e9291d628a994f21d59d > Cr-Commit-Position: refs/heads/master@{#33674} TBR=verwaest@chromium.org,ishell@chromium.org # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=chromium:579009 Review URL: https://codereview.chromium.org/1660263003 Cr-Commit-Position: refs/heads/master@{#33698}
-
neis authored
BUG=chromium:583260 LOG=n Review URL: https://codereview.chromium.org/1664683002 Cr-Commit-Position: refs/heads/master@{#33697}
-
hablich authored
Reland of [heap] Simplify distribution of remaining memory during sweeping & compaction (patchset #1 id:1 of https://codereview.chromium.org/1661723003/ ) Reason for revert: Bisection results show that this was not the culprit. Original issue's description: > Revert of [heap] Simplify distribution of remaining memory during sweeping & compaction (patchset #2 id:80001 of https://codereview.chromium.org/1653973003/ ) > > Reason for revert: > Very likely blocking roll: https://codereview.chromium.org/1652413003/ > > Original issue's description: > > [heap] Simplify distribution of remaining memory during sweeping & compaction > > > > BUG=chromium:524425 > > LOG=N > > > > Committed: https://crrev.com/f72923526ccaa8faef5c977267b0c074c4a44dfa > > Cr-Commit-Position: refs/heads/master@{#33668} > > TBR=hpayer@chromium.org,mlippautz@chromium.org > # Skipping CQ checks because original CL landed less than 1 days ago. > NOPRESUBMIT=true > NOTREECHECKS=true > NOTRY=true > BUG=chromium:524425 > > Committed: https://crrev.com/a9441b0e7a2a56c2047482a3cc66e3ca2255444b > Cr-Commit-Position: refs/heads/master@{#33695} TBR=hpayer@chromium.org,mlippautz@chromium.org # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=chromium:524425 Review URL: https://codereview.chromium.org/1663013002 Cr-Commit-Position: refs/heads/master@{#33696}
-
hablich authored
Revert of [heap] Simplify distribution of remaining memory during sweeping & compaction (patchset #2 id:80001 of https://codereview.chromium.org/1653973003/ ) Reason for revert: Very likely blocking roll: https://codereview.chromium.org/1652413003/ Original issue's description: > [heap] Simplify distribution of remaining memory during sweeping & compaction > > BUG=chromium:524425 > LOG=N > > Committed: https://crrev.com/f72923526ccaa8faef5c977267b0c074c4a44dfa > Cr-Commit-Position: refs/heads/master@{#33668} TBR=hpayer@chromium.org,mlippautz@chromium.org # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=chromium:524425 Review URL: https://codereview.chromium.org/1661723003 Cr-Commit-Position: refs/heads/master@{#33695}
-
bmeurer authored
We always call GetCallerArguments with 0 for prefix_argc, and so there's no use in having that parameter at all. R=yangguo@chromium.org Review URL: https://codereview.chromium.org/1662953002 Cr-Commit-Position: refs/heads/master@{#33694}
-
bmeurer authored
R=jarin@chromium.org BUG=chromium:582703 LOG=n Review URL: https://codereview.chromium.org/1664483003 Cr-Commit-Position: refs/heads/master@{#33693}
-