- 24 Jul, 2015 19 commits
-
-
bmeurer authored
Fix invalid tail calls and wrong context register. R=ishell@chromium.org Review URL: https://codereview.chromium.org/1257603004 Cr-Commit-Position: refs/heads/master@{#29844}
-
oth authored
BUG=v8:4280 LOG=N Review URL: https://codereview.chromium.org/1230753004 Cr-Commit-Position: refs/heads/master@{#29843}
-
yangguo authored
BUG=v8:4320 LOG=N R=mstarzinger@chromium.org Review URL: https://codereview.chromium.org/1255803004 Cr-Commit-Position: refs/heads/master@{#29842}
-
ulan authored
This function will be used later instead of HasLowAllocationRate to decide how many pages to compact. BUG=chromium:502247 LOG=NO Review URL: https://codereview.chromium.org/1254603002 Cr-Commit-Position: refs/heads/master@{#29841}
-
yangguo authored
R=jkummerow@chromium.org Review URL: https://codereview.chromium.org/1248443003 Cr-Commit-Position: refs/heads/master@{#29840}
-
chunyang.dai authored
port d6ee366d (r29834). original commit message: This is the initial round of optimizations for the LoadGlobalViaContextStub and StoreGlobalViaContextStub, basically turning them into platform code stubs to avoid the Crankshaft overhead in the fast case, and making the runtime interface cheaper. BUG= Review URL: https://codereview.chromium.org/1258513003 Cr-Commit-Position: refs/heads/master@{#29839}
-
chunyang.dai authored
port 7877c4e0 (r29814). original commit message: Adds basic support for generation of interpreter bytecode handler code snippets. The InterpreterAssembler class exposes a set of low level, interpreter specific operations which can be used to build a Turbofan graph. The Interpreter class generates a bytecode handler snippet for each bytecode by assembling operations using an InterpreterAssembler. Currently only two simple bytecodes are supported: LoadLiteral0 and Return. BUG= Review URL: https://codereview.chromium.org/1256783002 Cr-Commit-Position: refs/heads/master@{#29838}
-
chunyang.dai authored
port bc8041dc (r29811). original commit message: Previous to this CL, ICs used a slightly different code idiom to get to C++ code from generated code than runtime intrinsics, using an IC_Utility class that in essence provided exactly the same functionality as Runtime::FunctionForId, but in its own quirky way. This CL unifies the two mechanisms, folding IC_Utility away by making all IC entry points in C++ code, e.g. IC miss handlers, full-fledged runtime intrinsics. This makes it possible to eliminate a bunch of ad-hoc declarations and adapters that the IC system had to needlessly re-invent. As a bonus and the original reason for this yak-shave: IC-related C++ runtime functions are now callable from TurboFan. BUG= Review URL: https://codereview.chromium.org/1252903002 Cr-Commit-Position: refs/heads/master@{#29837}
-
chunyang.dai authored
port 3334b830 (r20813). original commit message: HydrogenCodeStubs consume stack arguments via descriptor. All of this is controlled by the CallDescriptor. It's simply the case that if you specify less registers than the function arity calls for, the rest are assumed to be on the stack. Bailout handlers accept these constant stack arguments too. BUG= Review URL: https://codereview.chromium.org/1258553002 Cr-Commit-Position: refs/heads/master@{#29836}
-
machenbach authored
Revert of Revert "In RegExp, lastIndex is read with ToLength, not ToInteger" (patchset #1 id:1 of https://codereview.chromium.org/1243053005/) Reason for revert: [Sheriff] This causes a breakage with custom snapshot (bisected locally): http://build.chromium.org/p/client.v8/builders/V8%20Linux64%20GC%20Stress%20-%20custom%20snapshot/builds/1190 Original issue's description: > Revert "In RegExp, lastIndex is read with ToLength, not ToInteger" > > $toLength is slow, causing a 3.8%-8% regression in the Octane RegExp > benchmark. Reverting this patch brings it back up. To make this change, > we'll need a faster implementation fo $toLength. > > BUG=chromium:513160 > LOG=Y > R=adamk > > Committed: https://crrev.com/477d651c6a978bdf34954048a235895c62dab0ac > Cr-Commit-Position: refs/heads/master@{#29830} TBR=adamk@chromium.org,littledan@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=chromium:513160 Review URL: https://codereview.chromium.org/1254723005 Cr-Commit-Position: refs/heads/master@{#29835}
-
bmeurer authored
This is the initial round of optimizations for the LoadGlobalViaContextStub and StoreGlobalViaContextStub, basically turning them into platform code stubs to avoid the Crankshaft overhead in the fast case, and making the runtime interface cheaper. R=ishell@chromium.org BUG=chromium:510694 LOG=n Review URL: https://codereview.chromium.org/1238143002 Cr-Commit-Position: refs/heads/master@{#29834}
-
Michael Achenbach authored
Cr-Commit-Position: refs/heads/master@{#29833}
-
Michael Achenbach authored
Cr-Commit-Position: refs/heads/master@{#29832}
-
chunyang.dai authored
port 26ffee2c (r29789). original commit message: It's just the same as StoreTransitionDescriptor. BUG= Review URL: https://codereview.chromium.org/1253573004 Cr-Commit-Position: refs/heads/master@{#29831}
-
littledan authored
$toLength is slow, causing a 3.8%-8% regression in the Octane RegExp benchmark. Reverting this patch brings it back up. To make this change, we'll need a faster implementation fo $toLength. BUG=chromium:513160 LOG=Y R=adamk Review URL: https://codereview.chromium.org/1243053005 Cr-Commit-Position: refs/heads/master@{#29830}
-
v8-autoroll authored
Rolling v8/tools/clang to 89e7efc0e2c12578ed7217fcf176ec1a4e567b2a TBR=machenbach@chromium.org Review URL: https://codereview.chromium.org/1256733002 Cr-Commit-Position: refs/heads/master@{#29829}
-
chunyang.dai authored
port 1f295980 (r29787). original commit message: This fixes a recent regression where the register holding the original receiver was pushed onto the stack before the internal frame within the CallStubInRecordCallTarget helper was created. That in turn confused the stack walker when allocations in these stubs failed. BUG= Review URL: https://codereview.chromium.org/1247493004 Cr-Commit-Position: refs/heads/master@{#29828}
-
pcc authored
BUG=chromium:513074 R=bmeurer@chromium.org LOG=n Review URL: https://codereview.chromium.org/1249273003 Cr-Commit-Position: refs/heads/master@{#29827}
-
bradnelson authored
Adding CHECKED_OUT_VERSION in test/simdjs so that the bots don't revert and redownload each time. LOG=N BUG=None TEST=None R=littledan@chromium.org,machenbach@chromium.org Review URL: https://codereview.chromium.org/1242863003 Cr-Commit-Position: refs/heads/master@{#29826}
-
- 23 Jul, 2015 21 commits
-
-
mostynb authored
Review URL: https://codereview.chromium.org/1251593009 Cr-Commit-Position: refs/heads/master@{#29825}
-
Adam Klein authored
TBR=yangguo@chromium.org Review URL: https://codereview.chromium.org/1246983004 . Cr-Commit-Position: refs/heads/master@{#29824}
-
mbrandy authored
Port 1f295980 Original commit message: This fixes a recent regression where the register holding the original receiver was pushed onto the stack before the internal frame within the CallStubInRecordCallTarget helper was created. That in turn confused the stack walker when allocations in these stubs failed. R=mstarzinger@chromium.org, dstence@us.ibm.com, michael_dawson@ca.ibm.com BUG=chromium:512711 LOG=N Review URL: https://codereview.chromium.org/1252493004 Cr-Commit-Position: refs/heads/master@{#29823}
-
mbrandy authored
Port 7877c4e0 Original commit message: Adds basic support for generation of interpreter bytecode handler code snippets. The InterpreterAssembler class exposes a set of low level, interpreter specific operations which can be used to build a Turbofan graph. The Interpreter class generates a bytecode handler snippet for each bytecode by assembling operations using an InterpreterAssembler. Currently only two simple bytecodes are supported: LoadLiteral0 and Return. R=rmcilroy@chromium.org, dstence@us.ibm.com, michael_dawson@ca.ibm.com BUG= Review URL: https://codereview.chromium.org/1250723003 Cr-Commit-Position: refs/heads/master@{#29822}
-
mbrandy authored
Port bc8041dc Original commit message: Previous to this CL, ICs used a slightly different code idiom to get to C++ code from generated code than runtime intrinsics, using an IC_Utility class that in essence provided exactly the same functionality as Runtime::FunctionForId, but in its own quirky way. This CL unifies the two mechanisms, folding IC_Utility away by making all IC entry points in C++ code, e.g. IC miss handlers, full-fledged runtime intrinsics. This makes it possible to eliminate a bunch of ad-hoc declarations and adapters that the IC system had to needlessly re-invent. As a bonus and the original reason for this yak-shave: IC-related C++ runtime functions are now callable from TurboFan. R=danno@chromium.org, dstence@us.ibm.com, michael_dawson@ca.ibm.com BUG= Review URL: https://codereview.chromium.org/1249433004 Cr-Commit-Position: refs/heads/master@{#29821}
-
mbrandy authored
Port 3334b830 Original commit message; All of this is controlled by the CallDescriptor. It's simply the case that if you specify less registers than the function arity calls for, the rest are assumed to be on the stack. Bailout handlers accept these constant stack arguments too. R=mvstanton@chromium.org, dstence@us.ibm.com, michael_dawson@ca.ibm.com BUG= Review URL: https://codereview.chromium.org/1256563002 Cr-Commit-Position: refs/heads/master@{#29820}
-
mbrandy authored
Port 26ffee2c Original commit message: It's just the same as StoreTransitionDescriptor. R=mvstanton@chromium.org, dstence@us.ibm.com, michael_dawson@ca.ibm.com BUG= Review URL: https://codereview.chromium.org/1246143004 Cr-Commit-Position: refs/heads/master@{#29819}
-
rmcilroy authored
Fix GCMole warning from r29814 (7877c4e0). BUG=v8:4280 LOG=N NOTREECHECKS=true TBR=machenbach,mstarzinger Review URL: https://codereview.chromium.org/1243893004 Cr-Commit-Position: refs/heads/master@{#29818}
-
mstarzinger authored
This mostly removes dead code and obsolete special cases from the RawMachineAssembler::MakeNode helper, that shouldn't be necessary anymore. R=rmcilroy@chromium.org Review URL: https://codereview.chromium.org/1243253005 Cr-Commit-Position: refs/heads/master@{#29817}
-
hichris123 authored
According to the ECMA spec, a 24th hour is allowed if the minutes, seconds, and milliseconds are all zero (i.e. it's midnight). Previously, we parsed the date correctly, however, we failed to account in all checks for the possibility of a 24th hour. This CL changes the check to allow a 24th hour if it's exactly midnight. BUG=chromium:174609 LOG=Y Review URL: https://codereview.chromium.org/1240093005 Cr-Commit-Position: refs/heads/master@{#29816}
-
rossberg authored
Gets rid of IsSimpleParameterList predicate. R=mstarzinger@chromium.org, caitpotter88@gmail.com BUG= Review URL: https://codereview.chromium.org/1251603004 Cr-Commit-Position: refs/heads/master@{#29815}
-
rmcilroy authored
Adds basic support for generation of interpreter bytecode handler code snippets. The InterpreterAssembler class exposes a set of low level, interpreter specific operations which can be used to build a Turbofan graph. The Interpreter class generates a bytecode handler snippet for each bytecode by assembling operations using an InterpreterAssembler. Currently only two simple bytecodes are supported: LoadLiteral0 and Return. BUG=v8:4280 LOG=N Review URL: https://codereview.chromium.org/1239793002 Cr-Commit-Position: refs/heads/master@{#29814}
-
mvstanton authored
All of this is controlled by the CallDescriptor. It's simply the case that if you specify less registers than the function arity calls for, the rest are assumed to be on the stack. Bailout handlers accept these constant stack arguments too. BUG= Review URL: https://codereview.chromium.org/1250563004 Cr-Commit-Position: refs/heads/master@{#29813}
-
rossberg authored
While at it, remove the notion of INTERNAL variables. @caitp: Took some parts from your CL, since I was blocked on the temp scope bug. R=mstarzinger@chromium.org BUG=512574 LOG=N Review URL: https://codereview.chromium.org/1250513004 Cr-Commit-Position: refs/heads/master@{#29812}
-
danno authored
Previous to this CL, ICs used a slightly different code idiom to get to C++ code from generated code than runtime intrinsics, using an IC_Utility class that in essence provided exactly the same functionality as Runtime::FunctionForId, but in its own quirky way. This CL unifies the two mechanisms, folding IC_Utility away by making all IC entry points in C++ code, e.g. IC miss handlers, full-fledged runtime intrinsics. This makes it possible to eliminate a bunch of ad-hoc declarations and adapters that the IC system had to needlessly re-invent. As a bonus and the original reason for this yak-shave: IC-related C++ runtime functions are now callable from TurboFan. Review URL: https://codereview.chromium.org/1248303002 Cr-Commit-Position: refs/heads/master@{#29811}
-
yangguo authored
Nobody seems to use it. A good alternative is rlwrap. R=jochen@chromium.org Review URL: https://codereview.chromium.org/1250223002 Cr-Commit-Position: refs/heads/master@{#29810}
-
machenbach authored
There are many test names in the v8 code base that prefix others, which makes it hard to only run those tests. BUG=chromium:511215 LOG=n NOTRY=true Review URL: https://codereview.chromium.org/1251363002 Cr-Commit-Position: refs/heads/master@{#29809}
-
jochen authored
BUG=v8:3996 R=jarin@chromium.org LOG=n Review URL: https://codereview.chromium.org/1249723005 Cr-Commit-Position: refs/heads/master@{#29808}
-
rossberg authored
In particular, rename FormalParameterParsingState and friends to FormalParameters etc. This should not change any logic, but is a preparatory CL for a bunch of follow-up fixes and clean-ups. R=mstarzinger@chromium.org BUG= Review URL: https://codereview.chromium.org/1247443004 Cr-Commit-Position: refs/heads/master@{#29807}
-
machenbach authored
BUG= Review URL: https://codereview.chromium.org/1253643003 Cr-Commit-Position: refs/heads/master@{#29806}
-
yangguo authored
R=mvstanton@chromium.org Committed: https://crrev.com/937d4efbf9c399339fdc8e041bec8e80baa7b58f Cr-Commit-Position: refs/heads/master@{#29798} Review URL: https://codereview.chromium.org/1255613002 Cr-Commit-Position: refs/heads/master@{#29805}
-