- 14 Jul, 2015 1 commit
-
-
littledan authored
Duplicate parameters are banned both overall in strict mode and also in arrow functions. Our error message for both cases blamed strict mode, which is confusing. This patch fixes the message to point to arrow functions as a possible source as well. R=wingo, adamk LOG=N Review URL: https://codereview.chromium.org/1236863008 Cr-Commit-Position: refs/heads/master@{#29662}
-
- 07 Jul, 2015 1 commit
-
-
arv authored
This flag is on by default but it will allow us to turn that off in favor of harmony-sloppy in the future. BUG=v8:3305, v8:2198 LOG=N R=littledan@chromium.org, rossberg@chromium.org Review URL: https://codereview.chromium.org/1218803006 Cr-Commit-Position: refs/heads/master@{#29526}
-
- 26 Jun, 2015 1 commit
-
-
arv authored
Move class tests to es6 directory BUG=v8:3330 LOG=N R=adamk Review URL: https://codereview.chromium.org/1213813003 Cr-Commit-Position: refs/heads/master@{#29336}
-
- 25 Jun, 2015 1 commit
-
-
wingo authored
BUG=v8:4213 R=arv@chromium.org LOG=N Review URL: https://codereview.chromium.org/1210003003 Cr-Commit-Position: refs/heads/master@{#29299}
-
- 24 Jun, 2015 1 commit
-
-
wingo authored
Some tokens need special messages because their token corresponds to many names. R=arv@chromium.org BUG=v8:4213 LOG=N Review URL: https://codereview.chromium.org/1207743004 Cr-Commit-Position: refs/heads/master@{#29262}
-
- 19 Jun, 2015 1 commit
-
-
wingo authored
R=rossberg@chromium.org BUG=v8:4194 LOG=N Review URL: https://codereview.chromium.org/1191303002 Cr-Commit-Position: refs/heads/master@{#29153}
-
- 16 Jun, 2015 1 commit
-
-
chunyang.dai authored
This patch includes the following changes. 1, Enable the turbofan backend support for X87 platform. It depends on previous CL: 3fdfebd2. 2, Enable the test cases which are disabled because turbofan for X87 was not enabled. BUG=v8:4135 LOG=N Review URL: https://codereview.chromium.org/1179763004 Cr-Commit-Position: refs/heads/master@{#29049}
-
- 12 Jun, 2015 1 commit
-
-
chunyang.dai authored
The strong-object-set-proto test cases will throw one expected exception and the exception message should include the source position in the JS file. But crankshaft compiler does not generate the source position for it when creating the flow graph by default. The source position information is always zero. So it failed when comparing with the reference file. If we use crankshaft compiler on IA32 platform to run this test case, it has the same failure. we will open these test case for x87 once turbofan for X87 is enabled. BUG= Review URL: https://codereview.chromium.org/1179763003 Cr-Commit-Position: refs/heads/master@{#29003}
-
- 10 Jun, 2015 1 commit
-
-
wingo authored
R=dslomov@chromium.org, rossberg@chromium.org LOG=Y BUG=v8:2700 Review URL: https://codereview.chromium.org/1178523002 Cr-Commit-Position: refs/heads/master@{#28908}
-
- 01 Jun, 2015 1 commit
-
-
conradw authored
A mistake in a recent CL has messed up the error messages for strong object semantics. BUG= LOG=N Review URL: https://codereview.chromium.org/1166433007 Cr-Commit-Position: refs/heads/master@{#28741}
-
- 18 May, 2015 2 commits
-
-
dslomov authored
R=arv@chromium.org,rossberg@chromium.org BUG=v8:811 LOG=N Review URL: https://codereview.chromium.org/1139773005 Cr-Commit-Position: refs/heads/master@{#28457}
-
caitpotter88 authored
BUG=v8:4107, v8:2159 LOG=N R=arv@chromium.org Review URL: https://codereview.chromium.org/1141223002 Cr-Commit-Position: refs/heads/master@{#28454}
-
- 21 Apr, 2015 1 commit
-
-
wingo authored
This commit is a precursor to making lazy arrow function parsing use similar logic to function(){} argument parsing. Originally landed in these three CLs: https://codereview.chromium.org/1078093002 https://codereview.chromium.org/1083623002 https://codereview.chromium.org/1083953002 These were rolled out due to a performance regression on CodeLoad. This patchset will fix that by avoiding creation of a DuplicateFinder in the full parser. R=marja@chromium.org BUG= LOG=N Review URL: https://codereview.chromium.org/1100713002 Cr-Commit-Position: refs/heads/master@{#27960}
-
- 17 Apr, 2015 1 commit
-
-
wingo authored
Revert https://codereview.chromium.org/1078093002/ and follow-on parser patches due to a perf regression. This reverts commit 53ddccfc. This reverts commit 71d3213a. This reverts commit 0f432ebb. This reverts commit 1dbc4327. R=marja@chromium.org Review URL: https://codereview.chromium.org/1094653002 Cr-Commit-Position: refs/heads/master@{#27912}
-
- 14 Apr, 2015 1 commit
-
-
chunyang.dai authored
The overwrite-builtins test cases will throw one expected exception and the exception message should include the source position in the JS file. But crankshaft compiler does not generate the source position for it when creating the flow graph by default. The source position information is always zero. So it failed when comparing with the reference file. If we use crankshaft compiler on IA32 platform to run this test case, it has the same failure. BUG= Review URL: https://codereview.chromium.org/1086503002 Cr-Commit-Position: refs/heads/master@{#27807}
-
- 13 Apr, 2015 2 commits
-
-
mstarzinger authored
This makes the compilers agree on the source position of a message generated by "throw new Error()", it points to the beginning of the throw directive. R=titzer@chromium.org TEST=message/regress/regress-3995 BUG=v8:3995 LOG=N Review URL: https://codereview.chromium.org/1049703002 Cr-Commit-Position: refs/heads/master@{#27775}
-
wingo authored
This commit is a precursor to making lazy arrow function parsing use similar logic to function(){} argument parsing. R=arv@chromium.org BUG=4020 LOG=N Review URL: https://codereview.chromium.org/1078093002 Cr-Commit-Position: refs/heads/master@{#27773}
-
- 07 Apr, 2015 1 commit
-
-
caitpotter88 authored
The ES6 grammar forbids the initialization of variable declarations in IterationStatements. This CL will report `for (var x = y in z)` as a SyntaxError in strict mode (as done in JSC). It is possible that this could break sites in sloppy mode, and so that change can wait. BUG= R= LOG=N Review URL: https://codereview.chromium.org/1033823002 Cr-Commit-Position: refs/heads/master@{#27639}
-
- 01 Apr, 2015 1 commit
-
-
kozyatinskiy authored
Revert of Correctly compute line numbers in functions from the function constructor. (patchset #5 id:80001 of https://codereview.chromium.org/701093003/) Reason for revert: Locations from New Function are broken in DevTools. Original issue's description: > Correctly compute line numbers in functions from the function constructor. > > R=aandrey@chromium.org > BUG=chromium:109362 > LOG=Y > > Committed: https://code.google.com/p/v8/source/detail?r=25289 TBR=aandrey@chromium.org,yangguo@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=chromium:109362 LOG=Y Review URL: https://codereview.chromium.org/1053563002 Cr-Commit-Position: refs/heads/master@{#27564}
-
- 16 Mar, 2015 1 commit
-
-
yurys authored
BUG=chromium:339474 LOG=Y Review URL: https://codereview.chromium.org/1002263004 Cr-Commit-Position: refs/heads/master@{#27205}
-
- 06 Mar, 2015 1 commit
-
-
caitpotter88 authored
BUG= LOG=N R=arv@chromium.org Review URL: https://codereview.chromium.org/982153003 Cr-Commit-Position: refs/heads/master@{#27051}
-
- 26 Feb, 2015 1 commit
-
-
adamk authored
This also adds a new VariableMode, IMPORT, which will be used to do appropriate binding for Import-declared Variables. Only named imports are handled for now. "import *" and default import syntaxes have had their TODOs adjusted to match the new code structure. BUG=v8:1569 LOG=n Review URL: https://codereview.chromium.org/948303004 Cr-Commit-Position: refs/heads/master@{#26895}
-
- 25 Feb, 2015 1 commit
-
-
adamk authored
The new logic ensures that the error messages are the same in the "import { <reserved word> }" and "import { foo as <reserved ord> }" cases. Also prepares ParseImportNames for returning both the import and local names to ParseImportClause. BUG=v8:1569 LOG=n Review URL: https://codereview.chromium.org/952863006 Cr-Commit-Position: refs/heads/master@{#26863}
-
- 19 Feb, 2015 1 commit
-
-
adamk authored
Add() becomes AddLocalExport, which takes an export_name and a local_name. New parsing tests exercise this. Also start generating exports for default exports (though this doesn't yet handle anonymous default exports). BUG=v8:1569 LOG=n Review URL: https://codereview.chromium.org/934323004 Cr-Commit-Position: refs/heads/master@{#26758}
-
- 12 Feb, 2015 3 commits
-
-
Erik Arvidsson authored
BUG=v8:3330 LOG=N R=dslomov@chromium.org Review URL: https://codereview.chromium.org/917933007 Cr-Commit-Position: refs/heads/master@{#26633}
-
Erik Arvidsson authored
BUG=v8:3885 LOG=N R=adamk@chromium.org, dslomov@chromium.org Review URL: https://codereview.chromium.org/916293004 Cr-Commit-Position: refs/heads/master@{#26632}
-
Dmitry Lomov authored
This CL fixes tests that no longer valid and also fixes two issues: 1. 'super()' in non derived constructors. 2. Failure to step into derived constructors. R=arv@chromium.org, yurys@chromium.org BUG=v8:3834 LOG=Y Review URL: https://codereview.chromium.org/923443003 Cr-Commit-Position: refs/heads/master@{#26628}
-
- 10 Dec, 2014 1 commit
-
-
dslomov authored
Change tests so that message location is stable under --debug-code. R=yangguo@chromium.org BUG=v8:3744 LOG=N Review URL: https://codereview.chromium.org/792563003 Cr-Commit-Position: refs/heads/master@{#25732}
-
- 08 Dec, 2014 2 commits
-
-
Dmitry Lomov authored
TBR=arv@chromium.org BUG=v8:3744 LOG=N NOTRY=true Review URL: https://codereview.chromium.org/746743003 Cr-Commit-Position: refs/heads/master@{#25716}
-
dslomov authored
R=arv@chromium.org BUG=v8:3330 LOG=N Review URL: https://codereview.chromium.org/776563002 Cr-Commit-Position: refs/heads/master@{#25714}
-
- 12 Nov, 2014 1 commit
-
-
yangguo@chromium.org authored
R=aandrey@chromium.org BUG=chromium:109362 LOG=Y Review URL: https://codereview.chromium.org/701093003 Cr-Commit-Position: refs/heads/master@{#25289} git-svn-id: https://v8.googlecode.com/svn/branches/bleeding_edge@25289 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
- 14 Aug, 2014 1 commit
-
-
mstarzinger@chromium.org authored
R=verwaest@chromium.org Review URL: https://codereview.chromium.org/473873002 git-svn-id: https://v8.googlecode.com/svn/branches/bleeding_edge@23130 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
- 12 Feb, 2014 2 commits
-
-
rmcilroy@chromium.org authored
- Remove arch specific check macros - Remove duplicate code in code-stubs.h - Remove flag check in bootstrap.cc which was introduced for A64 bringup - Remove A64 specific test message expectations R=rodolph.perfetta@arm.com, ulan@chromium.org Review URL: https://codereview.chromium.org/134333011 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@19325 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
ulan@chromium.org authored
BUG=v8:3113 LOG=Y R=jochen@chromium.org, rmcilroy@chromium.org, rodolph.perfetta@arm.com Review URL: https://codereview.chromium.org/148293020 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@19311 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
- 22 Nov, 2013 1 commit
-
-
yangguo@chromium.org authored
The changes are (excluding presubmit.py) mechanical. I added the following lines after the check and iterated the presubmit script until all errors went away: f = open(name, "w"); if contents.endswith('\n\n'): f.write(contents[0:-1]) else: f.write(contents + '\n') R=jkummerow@chromium.org Review URL: https://codereview.chromium.org/82803005 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@18017 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
- 08 Nov, 2013 1 commit
-
-
yangguo@chromium.org authored
R=bmeurer@chromium.org BUG=82797, v8:1593 Review URL: https://codereview.chromium.org/66463002 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17587 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
- 24 Oct, 2013 1 commit
-
-
mstarzinger@chromium.org authored
R=danno@chromium.org Review URL: https://codereview.chromium.org/39303004 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17381 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
- 23 Oct, 2013 1 commit
-
-
dslomov@chromium.org authored
Enable optimizing compiler for V8 built-ins. Also fixes an issue uncovered in x64 codegen. R=danno@chromium.org, mstarzinger@chromium.org, svenpanne@chromium.org Review URL: https://codereview.chromium.org/34503003 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17334 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
- 24 Sep, 2013 1 commit
-
-
jkummerow@chromium.org authored
R=machenbach@chromium.org Review URL: https://codereview.chromium.org/23498058 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@16919 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
- 26 Jun, 2013 1 commit
-
-
machenbach@chromium.org authored
This initial CL contains the new test suite code and two tests for demonstration. Other tests will follow in a separate CL. R=jkummerow@chromium.org Review URL: https://codereview.chromium.org/17260002 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@15339 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-