- 25 Feb, 2015 5 commits
-
-
arv authored
We don't need this since this case is now a SyntaxError. BUG=None LOG=N R=dslomov@chromium.org Review URL: https://codereview.chromium.org/957823002 Cr-Commit-Position: refs/heads/master@{#26864}
-
dcarney authored
R=verwaest@chromium.org BUG= Review URL: https://codereview.chromium.org/945313003 Cr-Commit-Position: refs/heads/master@{#26862}
-
dusan.milosavljevic authored
TEST=kraken-1.1 benchmark BUG= Review URL: https://codereview.chromium.org/952233002 Cr-Commit-Position: refs/heads/master@{#26861}
-
balazs.kilvady authored
BUG= Review URL: https://codereview.chromium.org/950523003 Cr-Commit-Position: refs/heads/master@{#26860}
-
akos.palfi authored
Port affcfaf4 BUG= Review URL: https://codereview.chromium.org/954863002 Cr-Commit-Position: refs/heads/master@{#26837}
-
- 23 Feb, 2015 2 commits
-
-
dcarney authored
R=verwaest@chromium.org BUG= Review URL: https://codereview.chromium.org/892383003 Cr-Commit-Position: refs/heads/master@{#26796}
-
dusan.milosavljevic authored
TEST= BUG= Review URL: https://codereview.chromium.org/945823002 Cr-Commit-Position: refs/heads/master@{#26793}
-
- 20 Feb, 2015 2 commits
-
-
balazs.kilvady authored
Also some target_at and target_at_put uniformed on mips and mips64. BUG= Review URL: https://codereview.chromium.org/942123002 Cr-Commit-Position: refs/heads/master@{#26785}
-
loislo authored
We accessed to cpu_profiler for tracking SharedFunctionInfo objects movements and used their addresses for generating function_id. Actually we could replace the manually generated shared_id by the pair script_id + position. In this case we can drop SharedFunctionInfo events support from cpu_profiler and remove the dependency. BTW GetCallUid was used as an unique identifier of the function on the front-end side. Actually it is a hash which might not be unique. So I renamed GetCallUid with GetHash and implemented GetFunctionId method. BUG=452067 LOG=n Review URL: https://codereview.chromium.org/941973002 Cr-Commit-Position: refs/heads/master@{#26775}
-
- 18 Feb, 2015 1 commit
-
-
adamk authored
ModuleDescriptor will end up holding the set of data described in the spec as a "Module record". This introduces a little bit of confusion with ModuleInfo, but I hope that'll become clearer over time. Also removed the interface-printing flags. We probably want Module-printing flags, but that can wait until we have more Module-related structures. BUG=v8:1569 LOG=n Review URL: https://codereview.chromium.org/935723004 Cr-Commit-Position: refs/heads/master@{#26728}
-
- 17 Feb, 2015 2 commits
-
-
adamk authored
This gets Variable and VariableProxy out of the business of worrying about Interfaces. At the same time, get rid of the notion of "module variables". In ES6, variables that refer to modules will be simply be CONST bindings to module namespace objects. The only change in logic here is one more early error: duplicate export names are now rejected. BUG=v8:1569 LOG=n Review URL: https://codereview.chromium.org/918373002 Cr-Commit-Position: refs/heads/master@{#26708}
-
balazs.kilvady authored
Port 49cbe537 BUG= Review URL: https://codereview.chromium.org/930623003 Cr-Commit-Position: refs/heads/master@{#26700}
-
- 14 Feb, 2015 1 commit
-
-
caitpotter88 authored
LOG=Y BUG=v8:2159 R=dslomov@chromium.org, arv@chromium.org, marja@chromium.org Review URL: https://codereview.chromium.org/816913003 Cr-Commit-Position: refs/heads/master@{#26645}
-
- 13 Feb, 2015 3 commits
-
-
loislo authored
BUG=none LOG=n Review URL: https://codereview.chromium.org/922153002 Cr-Commit-Position: refs/heads/master@{#26639}
-
Akos Palfi authored
Port 2707d4c9 Original commit message: 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. BUG= R=paul.lind@imgtec.com Review URL: https://codereview.chromium.org/920173002 Cr-Commit-Position: refs/heads/master@{#26635}
-
Akos Palfi authored
Port fdcf3e59 BUG= R=paul.lind@imgtec.com Review URL: https://codereview.chromium.org/920543004 Cr-Commit-Position: refs/heads/master@{#26634}
-
- 12 Feb, 2015 4 commits
-
-
Dmitry Lomov authored
R=arv@chromium.org BUG=v8:3834 LOG=N Review URL: https://codereview.chromium.org/919643008 Cr-Commit-Position: refs/heads/master@{#26631}
-
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 1 test 29 #3 2 opt_function 29 #4 2 opt_function 29 #5 deopted at 118 with reason 'not a heap number' deopted at 137 with reason 'division by zero' BUG=452067 LOG=n Committed: https://crrev.com/ce8701b247d3c6604f24f17a90c02d17b4417f54 Cr-Commit-Position: refs/heads/master@{#26615} Review URL: https://codereview.chromium.org/919953002 Cr-Commit-Position: refs/heads/master@{#26630}
-
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}
-
- 11 Feb, 2015 4 commits
-
-
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}
-
arv authored
We did not set up the [[HomeObject]] for properties created for computed property names. BUG=v8:3879 LOG=N R=dslomov@chromium.org Review URL: https://codereview.chromium.org/914773002 Cr-Commit-Position: refs/heads/master@{#26586}
-
arv authored
Support for `new super` has been removed so this code in the full code gen is now unreachable. BUG=None LOG=N R=dslomov@chromium.org Review URL: https://codereview.chromium.org/911223002 Cr-Commit-Position: refs/heads/master@{#26585}
-
- 10 Feb, 2015 3 commits
-
-
balazs.kilvady authored
Port 86cae163 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=452067 LOG=n Review URL: https://codereview.chromium.org/907383003 Cr-Commit-Position: refs/heads/master@{#26554}
-
balazs.kilvady authored
Port 5d68529b Original commit message: 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= Review URL: https://codereview.chromium.org/911073004 Cr-Commit-Position: refs/heads/master@{#26553}
-
loislo authored
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=452067 LOG=n Review URL: https://codereview.chromium.org/910773002 Cr-Commit-Position: refs/heads/master@{#26545}
-
- 05 Feb, 2015 8 commits
-
-
akos.palfi authored
Port 2491a639 BUG= Review URL: https://codereview.chromium.org/888143004 Cr-Commit-Position: refs/heads/master@{#26473}
-
Erik Arvidsson authored
This also removes some convenience functions that were not used. BUG=v8:3700 LOG=N R=adamk@chromium.org, adamk Review URL: https://codereview.chromium.org/883073008 Cr-Commit-Position: refs/heads/master@{#26472}
-
Dusan Milosavljevic authored
BUG= R=paul.lind@imgtec.com Review URL: https://codereview.chromium.org/905493002 Cr-Commit-Position: refs/heads/master@{#26469}
-
balazs.kilvady authored
Port 634b0f20 BUG= Review URL: https://codereview.chromium.org/900253002 Cr-Commit-Position: refs/heads/master@{#26467}
-
mvstanton authored
Looking up and passing this argument is unnecessary. R=ulan@chromium.org Review URL: https://codereview.chromium.org/898013003 Cr-Commit-Position: refs/heads/master@{#26464}
-
loislo authored
1) The hardcoded strings were converted into DeoptReason enum. 2) Deopt comment were converted into a pair location and deopt reason entries so the deopt reason tracking mode would less affect the size of the RelocInfo table and heap. 3) DeoptReason entry in RelocInfo reuses kCommentTag value and generates short entry in RelocInfo table. BUG=452067 LOG=n Committed: https://crrev.com/c49820e45b57f128a98690940875c049f612dde6 Cr-Commit-Position: refs/heads/master@{#26434} Committed: https://crrev.com/ec42e002da03adb2db968dd5b7453341ddc59a5c Cr-Commit-Position: refs/heads/master@{#26448} Review URL: https://codereview.chromium.org/874323003 Cr-Commit-Position: refs/heads/master@{#26463}
-
bmeurer authored
Revert of Externalize deoptimization reasons. (patchset #7 id:120001 of https://codereview.chromium.org/874323003/) Reason for revert: This CL breaks RelocInfo::INTERNAL_REFERENCE (and maybe others that come after DEOPT_REASON), which is currently (mostly) unused on tip-of-tree, but will be used for jump tables soon(ish) and should therefore work. It seems to be a problem with implicitly assumptions about the number of reloc info modes. Needs further investigation. Original issue's description: > Externalize deoptimization reasons. > > 1) The hardcoded strings were converted into DeoptReason enum. > > 2) Deopt comment were converted into a pair location and deopt reason entries so > the deopt reason tracking mode would less affect the size of the RelocInfo table and heap. > > 3) DeoptReason entry in RelocInfo reuses kCommentTag value and generates short entry in RelocInfo table. > > BUG=452067 > LOG=n > > Committed: https://crrev.com/c49820e45b57f128a98690940875c049f612dde6 > Cr-Commit-Position: refs/heads/master@{#26434} > > Committed: https://crrev.com/ec42e002da03adb2db968dd5b7453341ddc59a5c > Cr-Commit-Position: refs/heads/master@{#26448} TBR=alph@chromium.org,mstarzinger@chromium.org,svenpanne@chromium.org,yurys@chromium.org,loislo@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=452067 Review URL: https://codereview.chromium.org/900223002 Cr-Commit-Position: refs/heads/master@{#26457}
-
loislo authored
1) The hardcoded strings were converted into DeoptReason enum. 2) Deopt comment were converted into a pair location and deopt reason entries so the deopt reason tracking mode would less affect the size of the RelocInfo table and heap. 3) DeoptReason entry in RelocInfo reuses kCommentTag value and generates short entry in RelocInfo table. BUG=452067 LOG=n Committed: https://crrev.com/c49820e45b57f128a98690940875c049f612dde6 Cr-Commit-Position: refs/heads/master@{#26434} Review URL: https://codereview.chromium.org/874323003 Cr-Commit-Position: refs/heads/master@{#26448}
-
- 04 Feb, 2015 5 commits
-
-
paul.lind authored
Reason for revert: Fails mjsunit/asm/embenchen/box2d BUG= Review URL: https://codereview.chromium.org/893993003 Cr-Commit-Position: refs/heads/master@{#26444}
-
paul.lind authored
Port 13b97b62 Original commit message: On platforms that don't push the return address on the stack at time of call, there was an error. BUG= Review URL: https://codereview.chromium.org/864713003 Cr-Commit-Position: refs/heads/master@{#26443}
-
arv authored
We used to have a parse error for conflicting property keys. This check was removed to match ES6 (SpiderMonkey & Chakra already made this change). Since this check was removed we ended up with a few new cases when generating code. For example, accessors always generated code even if those should have been shadowed by a data property. BUG=v8:3856 LOG=Y R=adamk, dslomov@chromium.org Review URL: https://codereview.chromium.org/895693002 Cr-Commit-Position: refs/heads/master@{#26441}
-
balazs.kilvady authored
Port 275e088a BUG=chromium:455141 LOG=N Review URL: https://codereview.chromium.org/876083006 Cr-Commit-Position: refs/heads/master@{#26438}
-
arv authored
Class methods and accessors (both prototype and static) should be non enumerable. BUG=v8:3330 LOG=Y R=dslomov@chromium.org Review URL: https://codereview.chromium.org/896643003 Cr-Commit-Position: refs/heads/master@{#26437}
-