- 22 Oct, 2015 15 commits
-
-
neis authored
Proxies are not properly supported yet. R=rossberg BUG=v8:3931 LOG=n Review URL: https://codereview.chromium.org/1417243002 Cr-Commit-Position: refs/heads/master@{#31465}
-
ulan authored
This will allow us to remove background idle notification calls in Chrome. BUG=chromium:490559 LOG=NO Review URL: https://codereview.chromium.org/1406303008 Cr-Commit-Position: refs/heads/master@{#31464}
-
mlippautz authored
BUG= Review URL: https://codereview.chromium.org/1416003003 Cr-Commit-Position: refs/heads/master@{#31463}
-
Benedikt Meurer authored
Both the JSTypeFeedbackSpecializer and the JSTypeFeedbackLowering is dead code by now, since the more general JSNativeContextSpecialization deals with the property/global load/store type feedback in a way that also interacts properly with inlining. BUG=v8:4470 LOG=n R=jarin@chromium.org Review URL: https://codereview.chromium.org/1407913003 . Cr-Commit-Position: refs/heads/master@{#31462}
-
ulan authored
Incremental GC now finish quickly in background tabs too because they do not rely on idle notifications. BUG= Review URL: https://codereview.chromium.org/1423453003 Cr-Commit-Position: refs/heads/master@{#31461}
-
mlippautz authored
BUG=chromium:524425 LOG=N Review URL: https://codereview.chromium.org/1420803002 Cr-Commit-Position: refs/heads/master@{#31460}
-
ulan authored
BUG= Review URL: https://codereview.chromium.org/1419833002 Cr-Commit-Position: refs/heads/master@{#31459}
-
ishell authored
Remove support for "loads and stores to global vars through property cell shortcuts installed into parent script context" from all compilers. The plan is to implement the same idea using vector IC machinery. Stubs implementations and scopes modifications are left untouched for now. Review URL: https://codereview.chromium.org/1419823003 Cr-Commit-Position: refs/heads/master@{#31458}
-
jkummerow authored
NOTRY=true Review URL: https://codereview.chromium.org/1415863003 Cr-Commit-Position: refs/heads/master@{#31457}
-
bmeurer authored
Use a unified NamedAccess operator parameter for both JSLoadNamed and JSStoreNamed, and similar use PropertyAccess for both JSLoadProperty and JSStoreProperty. Review URL: https://codereview.chromium.org/1418993002 Cr-Commit-Position: refs/heads/master@{#31456}
-
bmeurer authored
R=jarin@chromium.org BUG=v8:4506 LOG=n Review URL: https://codereview.chromium.org/1413503006 Cr-Commit-Position: refs/heads/master@{#31455}
-
Michael Achenbach authored
Cr-Commit-Position: refs/heads/master@{#31454}
-
mtrofin authored
It appears we haven't wired in the 32 float constant passed on the stack. BUG=v8:4506 LOG=n Review URL: https://codereview.chromium.org/1407003015 Cr-Commit-Position: refs/heads/master@{#31453}
-
alph authored
BUG=v8:4406 LOG=N Review URL: https://codereview.chromium.org/1419983002 Cr-Commit-Position: refs/heads/master@{#31452}
-
v8-autoroll authored
Rolling v8/tools/swarming_client to 8fce79620b04bbe5415ace1103db27505bdc4c06 TBR=machenbach@chromium.org,vogelheim@chromium.org,hablich@chromium.org Review URL: https://codereview.chromium.org/1412173007 Cr-Commit-Position: refs/heads/master@{#31451}
-
- 21 Oct, 2015 24 commits
-
-
caitpotter88 authored
Add mechanism for adding per-context ECMAScript intrinsics to Template objects, which may be instantiated in multiple contexts. BUG= LOG=N R=jochen@chromium.org, verwaest@chromium.org Review URL: https://codereview.chromium.org/1409593002 Cr-Commit-Position: refs/heads/master@{#31450}
-
mbrandy authored
With --debug-code specified, 8 KB is not sufficient to contain some of the builtins on PPC64. This path is relevant only when snapshots are off. R=rmcilroy@chromium.org, ulan@chromium.org, michael_dawson@ca.ibm.com BUG= Review URL: https://codereview.chromium.org/1415463002 Cr-Commit-Position: refs/heads/master@{#31449}
-
mlippautz authored
R=ulan@chromium.org Review URL: https://codereview.chromium.org/1422543005 Cr-Commit-Position: refs/heads/master@{#31448}
-
neis authored
Proxies are not properly supported yet. R=rossberg BUG=v8:3931 LOG=n Review URL: https://codereview.chromium.org/1416433003 Cr-Commit-Position: refs/heads/master@{#31447}
-
vogelheim authored
(With a v8::Vector, the client is responsible for memory management. I think there can be a situation where the Vector has a char[1] backing store with '\0' in it, in which case the current code would leak. If we always Dispose() the backing store this should be avoided. Since dispose will delete[] the actual backing store, this should also work then the backing store is nullptr.) R=jochen@chromium.org BUG=chromium:525885 LOG=N Review URL: https://codereview.chromium.org/1410543005 Cr-Commit-Position: refs/heads/master@{#31446}
-
oth authored
This change adds new flavors of Visit() methods for obtaining expression results: - VisitForAccumulatorValue() which places result in the accumulator. - VisitForRegisterValue() which places the result in a register. - VisitForEffect() which evaluates the expression and discards the result. The targets of these calls place the expression result with result_scope()->SetResultInRegister() or result_scope()->SetResultInAccumulator(). By being smarter about result locations, there's less temporary register usage. However, we now have a hazard with assignments in binary expressions that didn't exist before. This change detects and DCHECK's when a hazard is detected. A follow on CL will address this. There are consequential changes to test-bytecode-generator.cc and this change also adds new bytecode macros A(x, n) and THIS(n) for register file entries for arguments and this. BUG=v8:4280 LOG=NO Review URL: https://codereview.chromium.org/1392933002 Cr-Commit-Position: refs/heads/master@{#31445}
-
jarin authored
The newly introduced root makes sure that we do not flush the optimized code while the function is being compiled. BUG=v8:4493 LOG=n Review URL: https://codereview.chromium.org/1415133002 Cr-Commit-Position: refs/heads/master@{#31444}
-
mlippautz authored
This reverts commit a31cef44. Original message: [heap] Divide available memory upon compaction tasks - Fairly (round-robin) divide available memory upon compaction tasks. - Ensure an upper limit (of memory) since dividing is O(n) for n free-space nodes. - Refill from free lists managed by sweeper once a compaction space becomes empty. Assumption for dividing memory: Memory in the free lists is sparse upon starting compaction (which means that only few nodes are available), except for memory reducer GCs, which happen in idle time though (so it's less of a problem). BUG=chromium:524425 LOG=N Review URL: https://codereview.chromium.org/1421583002 Cr-Commit-Position: refs/heads/master@{#31443}
-
machenbach authored
Set "no recover" in diagnostic mode to keep triggering crashes. Otherwise, the v8 test driver won't notice tests as failing. Also port some common sanitizer cflags to cfi - those can be removed on the build side afterwards. BUG=chromium:515782 LOG=n Review URL: https://codereview.chromium.org/1414763004 Cr-Commit-Position: refs/heads/master@{#31442}
-
adamk authored
R=mstarzinger@chromium.org Review URL: https://codereview.chromium.org/1418803002 Cr-Commit-Position: refs/heads/master@{#31441}
-
adamk authored
R=rossberg@chromium.org BUG=v8:4395 LOG=y Review URL: https://codereview.chromium.org/1414733005 Cr-Commit-Position: refs/heads/master@{#31440}
-
mlippautz authored
Revert of "[heap] Divide available memory upon compaction tasks" (patchset #5 id:90008 of https://codereview.chromium.org/1415733004/ ) Reason for revert: Failing again: http://build.chromium.org/p/client.v8/builders/V8%20Linux64%20GC%20Stress%20-%20custom%20snapshot/builds/2183 Original issue's description: > Reland of "[heap] Divide available memory upon compaction tasks" > > This reverts commit cf71c28f. > > Original message: > > [heap] Divide available memory upon compaction tasks > - Fairly (round-robin) divide available memory upon compaction tasks. > - Ensure an upper limit (of memory) since dividing is O(n) for n free-space > nodes. > - Refill from free lists managed by sweeper once a compaction space becomes > empty. > > Assumption for dividing memory: Memory in the free lists is sparse upon starting > compaction (which means that only few nodes are available), except for memory > reducer GCs, which happen in idle time though (so it's less of a problem). > > BUG=chromium:524425 > LOG=N > > Committed: https://crrev.com/63f42ecb965d04877f45043c1416170b6f79b962 > Cr-Commit-Position: refs/heads/master@{#31436} TBR=hpayer@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=chromium:524425 Review URL: https://codereview.chromium.org/1405273003 Cr-Commit-Position: refs/heads/master@{#31439}
-
ishell authored
Disable loads and stores to global vars through property cell shortcuts installed into parent script context. The plan is to implement the same idea using vector IC machinery. Review URL: https://codereview.chromium.org/1422463004 Cr-Commit-Position: refs/heads/master@{#31438}
-
mstarzinger authored
This makes sure that initializing assignments of let-declared variables perform an environment lookup and hence keep the variable alive. This is needed because full-codegen contains debug code verifying the variable is still inside the TDZ at the initializing assignment. R=jarin@chromium.org TEST=mjsunit/compiler/regress-variable-liveness-let BUG=v8:4493 LOG=n Review URL: https://codereview.chromium.org/1420573002 Cr-Commit-Position: refs/heads/master@{#31437}
-
mlippautz authored
This reverts commit cf71c28f. Original message: [heap] Divide available memory upon compaction tasks - Fairly (round-robin) divide available memory upon compaction tasks. - Ensure an upper limit (of memory) since dividing is O(n) for n free-space nodes. - Refill from free lists managed by sweeper once a compaction space becomes empty. Assumption for dividing memory: Memory in the free lists is sparse upon starting compaction (which means that only few nodes are available), except for memory reducer GCs, which happen in idle time though (so it's less of a problem). BUG=chromium:524425 LOG=N Review URL: https://codereview.chromium.org/1415733004 Cr-Commit-Position: refs/heads/master@{#31436}
-
adamk authored
When eagerly parsing arrow functions, expressions in default parameter initializers are parsed in the enclosing scope, rather than in the function's scope (since that scope does not yet exist). This leads to VariableProxies being added to the wrong scope, and scope chains for FunctionLiterals being incorrect. This patch addresses these problems by adding a subclass of AstExpressionVisitor that moves VariableProxies to the proper scope and fixes up scope chains of FunctionLiterals. This is a revert of the revert https://crrev.com/e41614a058426fb6102e4ab2dd4f98997f00c0fc with a much-improved (though not yet perfect) Scope::ResetOuterScope method which properly fixes not only the outer_scope_ pointer but also fixes the inner_scope_ list in the relevant outer_scopes. More work likely still needs to be done to make this work completely, but it's very close to correct. BUG=v8:4395 LOG=y Review URL: https://codereview.chromium.org/1414283002 Cr-Commit-Position: refs/heads/master@{#31435}
-
neis authored
Also clean up the access check, which was doing too much. This is in preparation of implementing Reflect.getPrototypeOf. BUG= Review URL: https://codereview.chromium.org/1402973002 Cr-Commit-Position: refs/heads/master@{#31434}
-
shiyu.zhang authored
port d515e513(r31211). contributed by zhengxing.li@intel.com original commit message: Previously, name conflicts between var and let declarations were only made into exceptions if they were visible at parse-time. This patch adds runtime checks so that sloppy-mode direct eval can't introduce conflicting var declarations. The change is implemented by traversing the scope chain when a direct eval introduces a var declaration to look for conflicting let declarations, up to the function boundary. additional comment: The original r31211 patch needs some additional changes for x87 1. Should use Smi::FromInt(variable->DeclarationPropertyAttributes()) BUG= Review URL: https://codereview.chromium.org/1420683002 Cr-Commit-Position: refs/heads/master@{#31433}
-
neis authored
This is in preparation of implementing Reflect.setPrototypeOf. R=verwaest@chromium.org BUG= Review URL: https://codereview.chromium.org/1409003005 Cr-Commit-Position: refs/heads/master@{#31432}
-
neis authored
Ignore proxies for now. R=rossberg BUG=v8:3931 LOG=n Review URL: https://codereview.chromium.org/1397853005 Cr-Commit-Position: refs/heads/master@{#31431}
-
mtrofin authored
Revert "Revert of [turbofan] Splinter into one range. (patchset #2 id:80001 of https://codereview.chromium.org/1391023007/ )" This reverts commit 23a8837f. Also added a CHECK in Merge to validate that splitting yields a different range and thus advances the algorithm. Ran stress bots successfully. Likely my earlier change in Splintering addressed the stress test scenario that was looping infinitely. BUG= Review URL: https://codereview.chromium.org/1406983004 Cr-Commit-Position: refs/heads/master@{#31430}
-
v8-autoroll authored
Rolling v8/build/gyp to 3f21260b43401553c37bb28090b65a75577d5b4e Rolling v8/tools/swarming_client to b64f0423ee66e730b0ceb88de8b9b89a98d7b61a TBR=machenbach@chromium.org,vogelheim@chromium.org,hablich@chromium.org Review URL: https://codereview.chromium.org/1417863002 Cr-Commit-Position: refs/heads/master@{#31429}
-
caitpotter88 authored
Adds an implementation of "do expression" parsing (https://webcache.googleusercontent.com/search?q=cache:MIGALjqPDNgJ:wiki.ecmascript.org/doku.php%3Fid%3Dstrawman:do_expressions+&cd=1&hl=en&ct=clnk&gl=us). This feature provides a way to evaluate a block of statements within an expression context, producing the resulting completion value. This is very helpful for implementing certain language features via desugaring. BUG=v8:4488 LOG=N R=adamk@chromium.org, bmeurer@chromium.org, rossberg@chromium.org, wingo@igalia.com Review URL: https://codereview.chromium.org/1399893002 Cr-Commit-Position: refs/heads/master@{#31428}
-
dpranke authored
This change defines a common config for clients of v8 that need to define V8_USE_EXTERNAL_STARTUP_DATA in order for client code to be compiled correctly. R=brettw@chromium.org, jochen@chromium.org BUG=541791 LOG=n Review URL: https://codereview.chromium.org/1415773003 Cr-Commit-Position: refs/heads/master@{#31427}
-
- 20 Oct, 2015 1 commit
-
-
dusan.m.milosavljevic authored
TEST= BUG= Review URL: https://codereview.chromium.org/1408033003 Cr-Commit-Position: refs/heads/master@{#31426}
-