- 20 Apr, 2015 1 commit
-
-
yangguo authored
R=jarin@chromium.org BUG=chromium:478011 LOG=N Review URL: https://codereview.chromium.org/1077153003 Cr-Commit-Position: refs/heads/master@{#27941}
-
- 17 Apr, 2015 1 commit
-
-
jkummerow authored
BUG=chromium:477924 LOG=n R=verwaest@chromium.org Review URL: https://codereview.chromium.org/1090193002 Cr-Commit-Position: refs/heads/master@{#27916}
-
- 15 Apr, 2015 3 commits
-
-
erikcorry authored
R=jkummerow@chromium.org BUG=chromium:475705 LOG=y Review URL: https://codereview.chromium.org/1082763002 Cr-Commit-Position: refs/heads/master@{#27851}
-
ulan authored
BUG=v8:4027 LOG=NO Review URL: https://codereview.chromium.org/1086063003 Cr-Commit-Position: refs/heads/master@{#27837}
-
jkummerow authored
This is a partial revert of 3eb277f2. R=machenbach@chromium.org NOTRY=true Review URL: https://codereview.chromium.org/1087183002 Cr-Commit-Position: refs/heads/master@{#27835}
-
- 14 Apr, 2015 2 commits
-
-
jkummerow authored
Review URL: https://codereview.chromium.org/1086923002 Cr-Commit-Position: refs/heads/master@{#27822}
-
mvstanton authored
BUG=476488 LOG=N R=jarin@chromium.org Review URL: https://codereview.chromium.org/1080253003 Cr-Commit-Position: refs/heads/master@{#27817}
-
- 13 Apr, 2015 2 commits
-
-
verwaest authored
BUG=chromium:388665 LOG=n Review URL: https://codereview.chromium.org/1087463003 Cr-Commit-Position: refs/heads/master@{#27795}
-
ulan authored
BUG=v8:4023 LOG=NO Review URL: https://codereview.chromium.org/1081033004 Cr-Commit-Position: refs/heads/master@{#27779}
-
- 09 Apr, 2015 1 commit
-
-
mstarzinger authored
This is a workaround to make the debugger happy about TurboFan frames when the debugger causes frame inspection. Note that this can happen because the debugger can be activated while there still are optimized TurboFan activations on the stack. R=ishell@chromium.org BUG=chromium:465298 TEST=mjsunit/regress/regress-crbug-465298 LOG=N Review URL: https://codereview.chromium.org/1074793003 Cr-Commit-Position: refs/heads/master@{#27717}
-
- 08 Apr, 2015 1 commit
-
-
yangguo authored
R=jarin@chromium.org BUG=chromium:474297 LOG=N Review URL: https://codereview.chromium.org/1062143002 Cr-Commit-Position: refs/heads/master@{#27659}
-
- 07 Apr, 2015 3 commits
-
-
jkummerow authored
The motivation is that we prefer to avoid creating internal properties, and we have a usable field on maps ("transitions", which is not used for prototype maps). This CL also ensures the invariant that prototype maps are never shared, even if they are in dictionary mode. Review URL: https://codereview.chromium.org/1033653002 Cr-Commit-Position: refs/heads/master@{#27617}
-
yangguo authored
R=mstarzinger@chromium.org BUG=chromium:469480 LOG=N Review URL: https://codereview.chromium.org/1034173002 Cr-Commit-Position: refs/heads/master@{#27615}
-
jkummerow authored
Optimistically pushing a lot of arguments can run into the stack limit of the process, at least on operating systems where this limit is close to the limit that V8 sets for itself. BUG=chromium:469768 LOG=y Review URL: https://codereview.chromium.org/1056913003 Cr-Commit-Position: refs/heads/master@{#27614}
-
- 01 Apr, 2015 3 commits
-
-
erikcorry authored
R=mstarzinger@chromium.org BUG=v8:3976 BUG=472504 LOG=y Review URL: https://codereview.chromium.org/1051833002 Cr-Commit-Position: refs/heads/master@{#27571}
-
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}
-
erikcorry authored
Revert of Fix JSON parser Handle leak (patchset #3 id:40001 of https://codereview.chromium.org/1041483004/) Reason for revert: Reverting due to JSOn parser failures Original issue's description: > Fix JSON parser Handle leak > > R=verwaest@chromium.org > BUG=v8:3976 > LOG=y > > Committed: https://crrev.com/1ec850383bb82f6d8bebc7416e5f50b649d1eeaa > Cr-Commit-Position: refs/heads/master@{#27512} TBR=verwaest@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=v8:3976 Review URL: https://codereview.chromium.org/1052593002 Cr-Commit-Position: refs/heads/master@{#27562}
-
- 30 Mar, 2015 4 commits
-
-
ishell authored
BUG=chromium:470804 LOG=Y Review URL: https://codereview.chromium.org/1033273005 Cr-Commit-Position: refs/heads/master@{#27528}
-
verwaest authored
BUG= Review URL: https://codereview.chromium.org/1034393002 Cr-Commit-Position: refs/heads/master@{#27519}
-
erikcorry authored
R=verwaest@chromium.org BUG=v8:3976 LOG=y Review URL: https://codereview.chromium.org/1041483004 Cr-Commit-Position: refs/heads/master@{#27512}
-
mvstanton authored
A bug allows JSObject literals with elements to have the elements in the boilerplate modified. BUG=466993 LOG=N Review URL: https://codereview.chromium.org/1037273002 Cr-Commit-Position: refs/heads/master@{#27511}
-
- 26 Mar, 2015 2 commits
-
-
yangguo authored
Revert of Revert of Debugger: deduplicate shared function info when setting script break points. (patchset #1 id:1 of https://codereview.chromium.org/999273003/) Reason for revert: Reland since the failure has been fixed in https://codereview.chromium.org/1035523005/ Original issue's description: > Revert of Debugger: deduplicate shared function info when setting script break points. (patchset #4 id:60001 of https://codereview.chromium.org/998253005/) > > Reason for revert: > Code caching failures. > > Original issue's description: > > Debugger: deduplicate shared function info when setting script break points. > > > > Also fix Debug.showBreakPoints for multiple break points at the same location. > > > > BUG=v8:3960 > > LOG=N > > > > Committed: https://crrev.com/73b17a71a22564c0b66d9aa7c00948c748f5b290 > > Cr-Commit-Position: refs/heads/master@{#27444} > > TBR=mstarzinger@chromium.org > NOPRESUBMIT=true > NOTREECHECKS=true > NOTRY=true > BUG=v8:3960 > > Committed: https://crrev.com/9b29d008dfcc00bf56be8040add1d2c5e404673b > Cr-Commit-Position: refs/heads/master@{#27448} TBR=mstarzinger@chromium.org BUG=v8:3960 LOG=N Review URL: https://codereview.chromium.org/1037013002 Cr-Commit-Position: refs/heads/master@{#27472}
-
yangguo authored
R=ulan@chromium.org Review URL: https://codereview.chromium.org/1005053004 Cr-Commit-Position: refs/heads/master@{#27464}
-
- 25 Mar, 2015 3 commits
-
-
yangguo authored
Revert of Debugger: deduplicate shared function info when setting script break points. (patchset #4 id:60001 of https://codereview.chromium.org/998253005/) Reason for revert: Code caching failures. Original issue's description: > Debugger: deduplicate shared function info when setting script break points. > > Also fix Debug.showBreakPoints for multiple break points at the same location. > > BUG=v8:3960 > LOG=N > > Committed: https://crrev.com/73b17a71a22564c0b66d9aa7c00948c748f5b290 > Cr-Commit-Position: refs/heads/master@{#27444} TBR=mstarzinger@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=v8:3960 Review URL: https://codereview.chromium.org/999273003 Cr-Commit-Position: refs/heads/master@{#27448}
-
yangguo authored
Also fix Debug.showBreakPoints for multiple break points at the same location. BUG=v8:3960 LOG=N Review URL: https://codereview.chromium.org/998253005 Cr-Commit-Position: refs/heads/master@{#27444}
-
mstarzinger authored
This switches full-codegen to no longer push and pop StackHandler markers onto the operand stack, but relies on a range-based handler table instead. We only use StackHandlers in JSEntryStubs to mark the transition from C to JS code. Note that this makes deoptimization and OSR from within any try-block work out of the box, makes the non-exception paths faster and should overall be neutral on the memory footprint (pros). On the other hand it makes the exception paths slower and actually throwing and exception more expensive (cons). R=yangguo@chromium.org TEST=cctest/test-run-jsexceptions/DeoptTry Review URL: https://codereview.chromium.org/1010883002 Cr-Commit-Position: refs/heads/master@{#27440}
-
- 24 Mar, 2015 3 commits
-
-
titzer authored
R=mstarzinger@chromium.org BUG= Review URL: https://codereview.chromium.org/1018513003 Cr-Commit-Position: refs/heads/master@{#27425}
-
erikcorry authored
Also introduce --trace-fragmentation-verbose, and fix --always-compact. R=ulan@chromium.org BUG=v8:3976 LOG=y Review URL: https://codereview.chromium.org/1024823002 Cr-Commit-Position: refs/heads/master@{#27414}
-
jarin authored
The test demonstrates a bad interaction between arguments object materialization, escape analysis and exception handling. We can return a wrong arguments object if we materialize arguments object (using f.arguments) and then throw around f's frame so that f does not clean up the materialized frame information (see the MaterializedObjectStore in deoptimizer.h/.cc). If we enter another function that has the same frame pointer and request an arguments object of (or lazily deoptimize) that function, we can get the materialized object of the original function. We should clean up the materialized object store when we unwind the stack. BUG=v8:3985 LOG=n Review URL: https://codereview.chromium.org/1032623003 Cr-Commit-Position: refs/heads/master@{#27406}
-
- 23 Mar, 2015 2 commits
-
-
verwaest authored
BUG= Review URL: https://codereview.chromium.org/1017263003 Cr-Commit-Position: refs/heads/master@{#27379}
-
titzer authored
R=jarin@chromium.org BUG=chromium:469605 LOG=Y Review URL: https://codereview.chromium.org/1030623003 Cr-Commit-Position: refs/heads/master@{#27366}
-
- 20 Mar, 2015 3 commits
-
-
Toon Verwaest authored
BUG=chromium:469148 LOG=y R=dcarney@chromium.org Review URL: https://codereview.chromium.org/1023123003 Cr-Commit-Position: refs/heads/master@{#27346}
-
rossberg authored
TBR=dslomov@chromium.org BUG= Review URL: https://codereview.chromium.org/1027693003 Cr-Commit-Position: refs/heads/master@{#27339}
-
aperez authored
The problem manifests itself when parsing manages to return something meaningful in the presence of a stack overflow. This happens because calling ParserBase::Next() will still return one valid token on stack overflow, before starting to return invalid tokens. Take the following input as example: a.map(v => v + 1); | | already next token parsed (which will be an invalid token (identifier) because of a stack overflow) The "v" may have been already parsed into a VariableProxy, then if a stack overflow occurs, next token will be an invalid token (instead of Token::ARROW), but the parser will return the VariableProxy. This always happens when lazy-parsing arrow functions, so the position in the input stream where the the arrow function code ends is known. This fix adds a check that ensures that parsing ended at the end position of the arrow function. BUG=465671 LOG=N Review URL: https://codereview.chromium.org/1023483003 Cr-Commit-Position: refs/heads/master@{#27325}
-
- 19 Mar, 2015 1 commit
-
-
yangguo authored
TBR=jkummerow@chromium.org BUG=v8:3969 LOG=N Review URL: https://codereview.chromium.org/1019223002 Cr-Commit-Position: refs/heads/master@{#27291}
-
- 17 Mar, 2015 2 commits
-
-
mstarzinger authored
This ensures that there is only one stub that deals with unwinding the stack. Having more than one place containing that logic is brittle and error prone, especially when it is a corner case only for RangeErrors. R=titzer@chromium.org TEST=mjsunit/regress/regress-crbug-467047 BUG=chromium:467047 LOG=N Review URL: https://codereview.chromium.org/1012103002 Cr-Commit-Position: refs/heads/master@{#27243}
-
mstarzinger authored
The evaluation order of receiver versus arguments is not properly defined by C++. This caused issues with Clang where the environment changed after the receiveing environment was already loaded. R=jarin@chromium.org BUG=chromium:467531 TEST=mjsunit/regress/regress-crbug-467531 LOG=N Review URL: https://codereview.chromium.org/1015683002 Cr-Commit-Position: refs/heads/master@{#27238}
-
- 16 Mar, 2015 1 commit
-
-
ishell authored
BUG=chromium:467481 LOG=Y Review URL: https://codereview.chromium.org/1009933002 Cr-Commit-Position: refs/heads/master@{#27223}
-
- 13 Mar, 2015 2 commits
-
-
arv authored
ES6 specs the function length property (it was not part of ES5) and it makes it configurable. BUG=v8:3045 LOG=N R=mstarzinger@chromium.org, adamk@chromium.org CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_chromium_rel_ng;tryserver.blink:linux_blink_rel Review URL: https://codereview.chromium.org/993073002 Cr-Commit-Position: refs/heads/master@{#27190}
-
dslomov authored
We have been shipping harmony scoping for 2 Chrome releases now (M41 and M42). Time to remove the flag. R=rossberg@chromium.org LOG=Y Review URL: https://codereview.chromium.org/1007783002 Cr-Commit-Position: refs/heads/master@{#27187}
-