- 10 Dec, 2015 3 commits
-
-
jarin authored
MachineType is now a class with two enum fields: - MachineRepresentation - MachineSemantic Both enums are usable on their own, and this change switches some places from using MachineType to use just MachineRepresentation. Most notably: - register allocator now uses just the representation. - Phi and Select nodes only refer to representations. Review URL: https://codereview.chromium.org/1513543003 Cr-Commit-Position: refs/heads/master@{#32738}
-
ahaas authored
This operator now provides a second output which indicates whether the conversion from float32 to int64 was successful or not. The second output returns 0 if the conversion fails, or something else if the conversion succeeds. The second output can be ignored, which means that the operator can be used the same as the original operator. I implement the new operator on x64, arm64, and mips64. @v8-ppc-ports, can you please take care of the ppc64 implementation of the second output? R=titzer@chromium.org, v8-arm-ports@googlegroups.com, v8-mips-ports@googlegroups.com Review URL: https://codereview.chromium.org/1504363002 Cr-Commit-Position: refs/heads/master@{#32737}
-
verwaest authored
BUG=v8:3900, v8:3931, v8:1543, v8:3330 LOG=n Review URL: https://codereview.chromium.org/1511893002 Cr-Commit-Position: refs/heads/master@{#32731}
-
- 09 Dec, 2015 24 commits
-
-
yangguo authored
R=machenbach@chromium.org BUG=v8:4588 LOG=N Review URL: https://codereview.chromium.org/1508583002 Cr-Commit-Position: refs/heads/master@{#32730}
-
adamk authored
This is in preparation for turning the flag off by default. BUG=chromium:552100 LOG=n Review URL: https://codereview.chromium.org/1511293002 Cr-Commit-Position: refs/heads/master@{#32729}
-
verwaest authored
BUG= Review URL: https://codereview.chromium.org/1510833003 Cr-Commit-Position: refs/heads/master@{#32722}
-
verwaest authored
Make Error.prototype.toString spec compliant; and fix various side-effect-free error printing methods R=yangguo@chromium.org LOG=n Committed: https://crrev.com/5dffa35350d0f57402806e6bd87a914e1d5933e4 Cr-Commit-Position: refs/heads/master@{#32695} Review URL: https://codereview.chromium.org/1507273002 Cr-Commit-Position: refs/heads/master@{#32720}
-
cbruni authored
LOG=N BUG=v8:1543 Review URL: https://codereview.chromium.org/1509603005 Cr-Commit-Position: refs/heads/master@{#32718}
-
ulan authored
Compaction of the array with maps happens lazily upon adding new maps. BUG= Review URL: https://codereview.chromium.org/1481953002 Cr-Commit-Position: refs/heads/master@{#32717}
-
vogelheim authored
- Except 2x applications of ForceSet, where I still need to debug why the expected replacement doesn't work as expected. - Ca. 3/4 of the file is covered. R=jochen@chromium.org BUG= Review URL: https://codereview.chromium.org/1516483002 Cr-Commit-Position: refs/heads/master@{#32714}
-
bmeurer authored
Creating a with context cannot lazy/eager deoptimize, so we don't need to pass a frame state. R=jarin@chromium.org Review URL: https://codereview.chromium.org/1508373002 Cr-Commit-Position: refs/heads/master@{#32711}
-
mythria authored
Adds implementation and tests for CreateObjectLiteral, CreateArrayLiteral and CreateRegExpLiteral to bytecode graph builder. Also changes these bytecodes to expect three operands instead of using accumulator to pass one of the operands. This is done to avoid looking into the earlier nodes to fetch operands in the bytecode graph builder. Also adds support for wide variant of these bytecodes to bytecode generator and bytecode graph builder. BUG=v8:4280 LOG=N Review URL: https://codereview.chromium.org/1503963002 Cr-Commit-Position: refs/heads/master@{#32710}
-
bmeurer authored
R=jarin@chromium.org Review URL: https://codereview.chromium.org/1515513002 Cr-Commit-Position: refs/heads/master@{#32709}
-
mythria authored
ToBoolean was used with conditional jumps. An earlier cl (https://codereview.chromium.org/1426913002/) merges jumps and ToBoolean into a single bytecode. So, we no longer need ToBoolean bytecode. BUG=v8:4280 LOG=N Review URL: https://codereview.chromium.org/1507683005 Cr-Commit-Position: refs/heads/master@{#32707}
-
mlippautz authored
* Move most heap related tests into heap/ subdir * IWYU for heap utility functions R=ulan@chromium.org BUG= Review URL: https://codereview.chromium.org/1512553002 Cr-Commit-Position: refs/heads/master@{#32706}
-
ahaas authored
This operator now provides a second output which indicates whether the conversion from float64 to uint64 was successful or not. The second output returns 0 if the conversion fails, or something else if the conversion succeeds. The second output can be ignored, which means that the operator can be used the same as the original operator. I implement the new operator on x64 and arm64. @v8-mips-ports and @v8-ppc-ports, can you please take care of the mips64 and ppc64 implementation of the second output? R=titzer@chromium.org, v8-arm-ports@googlegroups.com Review URL: https://codereview.chromium.org/1507703002 Cr-Commit-Position: refs/heads/master@{#32705}
-
machenbach authored
NOTREECHECKS=true NOTRY=true NOPRESUBMIT=true TBR=verwaest@chromium.org Review URL: https://codereview.chromium.org/1510193002 Cr-Commit-Position: refs/heads/master@{#32703}
-
jochen authored
Embedders still can use those APIs by default test-api.cc still has an exception to use the old APIs... BUG=v8:4143 R=vogelheim@chromium.org LOG=n Review URL: https://codereview.chromium.org/1505803004 Cr-Commit-Position: refs/heads/master@{#32701}
-
machenbach authored
Revert of Make Error.prototype.toString spec compliant; and fix various side-effect-free error printing metho… (patchset #2 id:20001 of https://codereview.chromium.org/1507273002/ ) Reason for revert: [Sheriff] Breaks layout tests. Please rebase upstream first: https://build.chromium.org/p/client.v8.fyi/builders/V8-Blink%20Linux%2064/builds/3334 Original issue's description: > Make Error.prototype.toString spec compliant; and fix various side-effect-free error printing methods > > R=yangguo@chromium.org > LOG=n > > Committed: https://crrev.com/5dffa35350d0f57402806e6bd87a914e1d5933e4 > Cr-Commit-Position: refs/heads/master@{#32695} TBR=yangguo@chromium.org,bmeurer@chromium.org,verwaest@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true Review URL: https://codereview.chromium.org/1510173002 Cr-Commit-Position: refs/heads/master@{#32700}
-
jkummerow authored
JSProxy::HasProperty was missing an early "return Nothing<bool>". KeyAccumulator's FilterProxyKeys() didn't handle SKIP_STRINGS correctly. BUG=v8:1543 LOG=n R=cbruni@chromium.org Review URL: https://codereview.chromium.org/1505253002 Cr-Commit-Position: refs/heads/master@{#32699}
-
ishell authored
Review URL: https://codereview.chromium.org/1506683004 Cr-Commit-Position: refs/heads/master@{#32698}
-
mvstanton authored
It's expensive to walk all shared function infos during the gc atomic pause. Instead, use WeakCells to implement this structure without manual clearing. Reland due to a bug when reusing entries in the optimized code map. BUG= Review URL: https://codereview.chromium.org/1508703002 Cr-Commit-Position: refs/heads/master@{#32696}
-
verwaest authored
Make Error.prototype.toString spec compliant; and fix various side-effect-free error printing methods R=yangguo@chromium.org LOG=n Review URL: https://codereview.chromium.org/1507273002 Cr-Commit-Position: refs/heads/master@{#32695}
-
mvstanton authored
It's cumbersome to maintain IC profiler statistics all the time. Let's just do it as needed. BUG= Review URL: https://codereview.chromium.org/1507903004 Cr-Commit-Position: refs/heads/master@{#32693}
-
verwaest authored
BUG=v8:3900, v8:3931, v8:1543, v8:3330, v8:4002 LOG=n Review URL: https://codereview.chromium.org/1496333002 Cr-Commit-Position: refs/heads/master@{#32692}
-
yangguo authored
R=littledan@chromium.org BUG=v8:4342 LOG=N Review URL: https://codereview.chromium.org/1506173002 Cr-Commit-Position: refs/heads/master@{#32689}
-
bradnelson authored
Reconstructing which stdlib object is used within an asm.js module seems wasteful, given the typer has already checked this. Preserving this information in a form that can be queried in the asm-wasm-builder which generating code. BUG= https://code.google.com/p/v8/issues/detail?id=4203 TEST=test-asm-validator, test-parsing R=titzer@chromium.org,marja@chromium.org,aseemgarg@chromium.org LOG=N Review URL: https://codereview.chromium.org/1508003002 Cr-Commit-Position: refs/heads/master@{#32687}
-
- 08 Dec, 2015 8 commits
-
-
machenbach authored
NOTRY=true TBR=rmcilroy, Hablich Review URL: https://codereview.chromium.org/1510003002 Cr-Commit-Position: refs/heads/master@{#32683}
-
caitpotter88 authored
BUG=v8:3983 LOG=N R=wingo@igalia.com, littledan@chromium.org Review URL: https://codereview.chromium.org/1437803004 Cr-Commit-Position: refs/heads/master@{#32682}
-
jochen authored
BUG=v8:4134 R=vogelheim@chromium.org LOG=n NOTRY=true NOPRESUBMIT=true Review URL: https://codereview.chromium.org/1505173002 Cr-Commit-Position: refs/heads/master@{#32678}
-
jochen authored
BUG=v8:4134 R=vogelheim@chromium.org LOG=n NOTRY=true NOPRESUBMIT=true Review URL: https://codereview.chromium.org/1504093003 Cr-Commit-Position: refs/heads/master@{#32677}
-
jochen authored
BUG=v8:4341 R=vogelheim@chromium.org LOG=y NOTRY=true NOPRESUBMIT=true Review URL: https://codereview.chromium.org/1502353004 Cr-Commit-Position: refs/heads/master@{#32676}
-
cbruni authored
BUG=v8:1543 LOG=N Review URL: https://codereview.chromium.org/1499593003 Cr-Commit-Position: refs/heads/master@{#32675}
-
yangguo authored
R=jkummerow@chromium.org Review URL: https://codereview.chromium.org/1508653002 Cr-Commit-Position: refs/heads/master@{#32671}
-
littledan authored
ECMAScript dates act as if the Gregorian changeover happened at the beginning of time. This patch fixes up internationalized date formatting to set that changeover properly, as opposed to the ICU default which is in the 16th century. BUG=chromium:537382 R=adamk,cira LOG=Y Review URL: https://codereview.chromium.org/1501113002 Cr-Commit-Position: refs/heads/master@{#32669}
-
- 07 Dec, 2015 5 commits
-
-
dusan.m.milosavljevic authored
TEST=unittests/InstructionSlectorTest.CombineChangeFloat64ToInt32WithRoundFloat64 BUG= Review URL: https://codereview.chromium.org/1510493002 Cr-Commit-Position: refs/heads/master@{#32668}
-
verwaest authored
Error still to be done, since that's not yet available in the bootstrapper. BUG=v8:3900, v8:3931, v8:1543, v8:3330 LOG=n Review URL: https://codereview.chromium.org/1499923002 Cr-Commit-Position: refs/heads/master@{#32662}
-
neis authored
R=rossberg BUG= Review URL: https://codereview.chromium.org/1502983002 Cr-Commit-Position: refs/heads/master@{#32660}
-
jochen authored
BUG=v8:4134 R=vogelheim@chromium.org LOG=n Review URL: https://codereview.chromium.org/1502193002 Cr-Commit-Position: refs/heads/master@{#32659}
-
sigurds authored
This patch improves escape analysis and fixes bugs triggered by clusterfuzz. Impovements include: * Handling of LoadElement/StoreElement if index is a constant * Handling of JSStoreProperty: invalidate all information, as the store could have altered any field. * Treat phis that use an allocation as escaping * Improve resolution of replacements R=mstarzinger@chromium.org BUG=v8:4586 LOG=n Review URL: https://codereview.chromium.org/1499143002 Cr-Commit-Position: refs/heads/master@{#32656}
-