- 02 Jun, 2015 27 commits
-
-
bbudge authored
Adds SIMD 128 alignment sizes and masks. Adds support in Heap for SIMD alignments and fills. Reworks cctest so that each test independently aligns its allocation address, rather than depending on the previous tests ending state. Adds test cases for SIMD. LOG=N BUG=v8:4124 Committed: https://crrev.com/4347d56a6919ae06a70e4a4a8b2f1179cf47bc7e Cr-Commit-Position: refs/heads/master@{#28767} Review URL: https://codereview.chromium.org/1159453004 Cr-Commit-Position: refs/heads/master@{#28771}
-
mbrandy authored
Embed constant pools within their corresponding Code objects. This removes support for out-of-line constant pools in favor of the new approach -- the main advantage being that it eliminates the need to allocate and manage separate constant pool array objects. Currently supported on PPC and ARM. Enabled by default on PPC only. This yields a 6% improvment in Octane on PPC64. R=danno@chromium.org, svenpanne@chromium.org, bmeurer@chromium.org, rmcilroy@chromium.org, dstence@us.ibm.com, michael_dawson@ca.ibm.com BUG=chromium:478811 LOG=Y Review URL: https://codereview.chromium.org/1131783003 Cr-Commit-Position: refs/heads/master@{#28770}
-
arv authored
This splits the SuperReference AST node into SuperPropertyReference and SuperCallReference. The super call reference node consists of three unresolved vars to this, new.target and this_function. These gets declared when the right function is entered and if it is in use. The variables gets assigned in FullCodeGenerator::Generate. This is a revert of the revert 88b1c917 BUG=v8:3768 LOG=N R=wingo@igalia.com, adamk@chromium.org Review URL: https://codereview.chromium.org/1168513004 Cr-Commit-Position: refs/heads/master@{#28769}
-
bbudge authored
Revert of Add SIMD 128 alignment support to Heap. (patchset #3 id:40001 of https://codereview.chromium.org/1159453004/) Reason for revert: Breaks Linux - arm64 - sim - MSAN TBR=jochen Original issue's description: > Add SIMD 128 alignment support to Heap. > Adds SIMD 128 alignment sizes and masks. > Adds support in Heap for SIMD alignments and fills. > Reworks cctest so that each test independently aligns its allocation address, rather than depending on the previous tests ending state. Adds test cases for SIMD. > > LOG=N > BUG=v8:4124 TBR=hpayer@chromium.org,jochen@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=v8:4124 Review URL: https://codereview.chromium.org/1169453003 Cr-Commit-Position: refs/heads/master@{#28768}
-
bbudge authored
Adds SIMD 128 alignment sizes and masks. Adds support in Heap for SIMD alignments and fills. Reworks cctest so that each test independently aligns its allocation address, rather than depending on the previous tests ending state. Adds test cases for SIMD. LOG=N BUG=v8:4124 Review URL: https://codereview.chromium.org/1159453004 Cr-Commit-Position: refs/heads/master@{#28767}
-
dehrenberg authored
Implementations factored out from Array. Tests are derived from normal array toString tests. BUG=v8:3578 LOG=Y R=adamk Review URL: https://codereview.chromium.org/1166623004 Cr-Commit-Position: refs/heads/master@{#28766}
-
mbrandy authored
R=dstence@us.ibm.com, michael_dawson@ca.ibm.com BUG= Review URL: https://codereview.chromium.org/1156823003 Cr-Commit-Position: refs/heads/master@{#28765}
-
mbrandy authored
Port 450002f3 Original commit message: It's necessary to do this in order to know how many type feedback vector slots we should allocate for the object literal. R=mvstanton@chromium.org, dstence@us.ibm.com, michael_dawson@ca.ibm.com BUG= Review URL: https://codereview.chromium.org/1166473004 Cr-Commit-Position: refs/heads/master@{#28764}
-
titzer authored
R=mstarzinger@chromium.org BUG= Review URL: https://codereview.chromium.org/1146173005 Cr-Commit-Position: refs/heads/master@{#28763}
-
verwaest authored
BUG=v8:4137 LOG=n Review URL: https://codereview.chromium.org/1157093003 Cr-Commit-Position: refs/heads/master@{#28762}
-
mstarzinger authored
R=bmeurer@chromium.org BUG=v8:4158 LOG=N Review URL: https://codereview.chromium.org/1158883006 Cr-Commit-Position: refs/heads/master@{#28761}
-
conradw authored
Copied, with permission, from https://codereview.chromium.org/1151853003/ Initial patch set is an unmodified copy, rebased on top of related fixes from https://codereview.chromium.org/1158933002/ Subsequent patch sets contain fixes for remaining bugs in the CL. BUG=v8:3956 LOG=N Review URL: https://codereview.chromium.org/1152093003 Cr-Commit-Position: refs/heads/master@{#28760}
-
ishell authored
BUG=chromium:493568 LOG=N Review URL: https://codereview.chromium.org/1158023003 Cr-Commit-Position: refs/heads/master@{#28759}
-
verwaest authored
BUG= Review URL: https://codereview.chromium.org/1143153004 Cr-Commit-Position: refs/heads/master@{#28758}
-
verwaest authored
BUG=v8:4137 LOG=n Review URL: https://codereview.chromium.org/1153583006 Cr-Commit-Position: refs/heads/master@{#28757}
-
ishell authored
This is just a workaround till we found the root cause of the issue, there must be no slots for data object recorded. BUG=chromium:454297 LOG=N Review URL: https://codereview.chromium.org/1106983004 Cr-Commit-Position: refs/heads/master@{#28756}
-
martyn.capewell authored
Select ubfiz for (x & mask) << imm where mask is contiguous and imm is non-zero. BUG= Review URL: https://codereview.chromium.org/1161643003 Cr-Commit-Position: refs/heads/master@{#28755}
-
verwaest authored
This also fixes issues with - kMaxUint32 being a valid length but not index cornercases - exotic integer objects masking "exotic indexes" even though its in the prototype chain - concating of holey sloppy arguments BUG=v8:4137 LOG=n Review URL: https://codereview.chromium.org/1159433003 Cr-Commit-Position: refs/heads/master@{#28754}
-
mstarzinger authored
R=bmeurer@chromium.org BUG=v8:4131 LOG=N Review URL: https://codereview.chromium.org/1152173011 Cr-Commit-Position: refs/heads/master@{#28753}
-
ishell authored
BUG= Review URL: https://codereview.chromium.org/1156993018 Cr-Commit-Position: refs/heads/master@{#28752}
-
rossberg authored
R=bmeurer@chromium.org BUG=chromium:492022,v8:3956 LOG=N Review URL: https://codereview.chromium.org/1159623008 Cr-Commit-Position: refs/heads/master@{#28751}
-
mstarzinger authored
This adds handling of JSLoadDynamicGlobal nodes to JSTypedLowering to perform extension checks and an inline fast path. The fast path is a global variable load from the global object. R=bmeurer@chromium.org BUG=v8:4131 LOG=N Review URL: https://codereview.chromium.org/1150723005 Cr-Commit-Position: refs/heads/master@{#28750}
-
mvstanton authored
It's necessary to do this in order to know how many type feedback vector slots we should allocate for the object literal. BUG= Review URL: https://codereview.chromium.org/1165463007 Cr-Commit-Position: refs/heads/master@{#28749}
-
chunyang.dai authored
port e2e47f30 (r28711) original commit message: [turbofan] First step towards sanitizing for-in and making it optimizable. In a nutshell: The FILTER_KEY builtin is gone, and was replaced by a simple runtime call to ForInFilter, which does everything and is even cheaper (because FILTER_KEY used to call into the runtime anyway). And ForInFilter returns either the name or undefined, which makes it possible to remove the control flow construction from the AstGraphBuilder, and thereby make both the initialization and the per-loop code of for-in optimizable later (in typed lowering). BUG= Review URL: https://codereview.chromium.org/1144143005 Cr-Commit-Position: refs/heads/master@{#28748}
-
bmeurer authored
Unfortunately StringAdd is not pure in V8 because we might throw an exception if the resulting string length is outside the valid bounds, so there's no point in having a simplified StringAdd operator. R=jarin@chromium.org Review URL: https://codereview.chromium.org/1164743002 Cr-Commit-Position: refs/heads/master@{#28747}
-
bmeurer authored
This avoids redundant smi checks when using the length of a FixedArray, which is always a positve smi. R=jarin@chromium.org Review URL: https://codereview.chromium.org/1162163003 Cr-Commit-Position: refs/heads/master@{#28746}
-
v8-autoroll authored
Rolling v8/tools/clang to dc542e2b87607a399e0c107b78990aba11f5af48 TBR=machenbach@chromium.org Review URL: https://codereview.chromium.org/1143213004 Cr-Commit-Position: refs/heads/master@{#28745}
-
- 01 Jun, 2015 13 commits
-
-
arv authored
BUG=v8:3018 LOG=N Review URL: https://codereview.chromium.org/1165653003 Cr-Commit-Position: refs/heads/master@{#28744}
-
adamk authored
The Map and Set maps get overwritten when collection.js executes, so in a nosnap build we have to wait until it runs before we grab the maps. To facilitate that, store the functions in the native context as well. Review URL: https://codereview.chromium.org/1161363002 Cr-Commit-Position: refs/heads/master@{#28743}
-
erikcorry authored
When compiling on a laptop I like to concatenate the small test files. This makes a big difference to compile times. These changes make that easier. R=ulan@chromium.org BUG= Review URL: https://codereview.chromium.org/1163803002 Cr-Commit-Position: refs/heads/master@{#28742}
-
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}
-
caitpotter88 authored
Revert of [es6] implement default parameters via desugaring (patchset #19 id:380001 of https://codereview.chromium.org/1127063003/) Reason for revert: Broken on arm64 Original issue's description: > [es6] implement default parameters via desugaring > > Stage 1 implementation: > > - Parameters can't be referenced before initialized (from left-to-right) > - SingleNameBindings only, no support for BindingPatterns > > Known issues: > > - Incorrect scoping (parameter expressions may reference variables declared in function body) > - Function arity is untouched > - Hole-checking needs work > - Rest parameters are broken when mixed with optional arguments > > BUG=v8:2160 > LOG=N > R=arv@chromium.org, rossberg@chromium.org > > Committed: https://crrev.com/892c85485881f8be2f17bd83238980f858126576 > Cr-Commit-Position: refs/heads/master@{#28739} TBR=rossberg@chromium.org,wingo@igalia.com,arv@chromium.org,dslomov@chromium.org,adamk@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=v8:2160 Review URL: https://codereview.chromium.org/1163853002 Cr-Commit-Position: refs/heads/master@{#28740}
-
caitpotter88 authored
Stage 1 implementation: - Parameters can't be referenced before initialized (from left-to-right) - SingleNameBindings only, no support for BindingPatterns Known issues: - Incorrect scoping (parameter expressions may reference variables declared in function body) - Function arity is untouched - Hole-checking needs work - Rest parameters are broken when mixed with optional arguments BUG=v8:2160 LOG=N R=arv@chromium.org, rossberg@chromium.org Review URL: https://codereview.chromium.org/1127063003 Cr-Commit-Position: refs/heads/master@{#28739}
-
ishell authored
Dictionary customization should be implemented in respective shape class. Review URL: https://codereview.chromium.org/1160813009 Cr-Commit-Position: refs/heads/master@{#28738}
-
ishell authored
Review URL: https://codereview.chromium.org/1165603002 Cr-Commit-Position: refs/heads/master@{#28737}
-
nodir authored
Added /infra with chrome-infra specific files. /infra/project-config/cr-buildbucket.cfg defines v8 buckets on cr-buildbucket.appspot.com R=tandrii@chromium.org, machenbach@chromium.org, sergiyb@chromium.org BUG= Review URL: https://codereview.chromium.org/1161143002 Cr-Commit-Position: refs/heads/master@{#28736}
-
arv authored
Revert of [es6] Super call in arrows and eval (patchset #5 id:100001 of https://codereview.chromium.org/1146863007/) Reason for revert: Fails http://build.chromium.org/p/client.v8/builders/V8%20Linux%20-%20nosnap%20-%20debug%20-%201/builds/579/steps/Check/logs/super Original issue's description: > [es6] Super call in arrows and eval > > This splits the SuperReference AST node into SuperPropertyReference and > SuperCallReference. The super call reference node consists of three > unresolved vars to this, new.target and this_function. These gets > declared when the right function is entered and if it is in use. The > variables gets assigned in FullCodeGenerator::Generate. > > BUG=v8:3768 > LOG=N > R=wingo@igalia.com, adamk@chromium.org > > Committed: https://crrev.com/673c0516ab96f24343bbb26e0afc2846b5a679df > Cr-Commit-Position: refs/heads/master@{#28731} TBR=wingo@igalia.com,adamk@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=v8:3768 Review URL: https://codereview.chromium.org/1161243005 Cr-Commit-Position: refs/heads/master@{#28735}
-
mbrandy authored
Port e2e47f30 Original commit message: In a nutshell: The FILTER_KEY builtin is gone, and was replaced by a simple runtime call to ForInFilter, which does everything and is even cheaper (because FILTER_KEY used to call into the runtime anyway). And ForInFilter returns either the name or undefined, which makes it possible to remove the control flow construction from the AstGraphBuilder, and thereby make both the initialization and the per-loop code of for-in optimizable later (in typed lowering). R=bmeurer@chromium.org, dstence@us.ibm.com, michael_dawson@ca.ibm.com BUG= Review URL: https://codereview.chromium.org/1160973004 Cr-Commit-Position: refs/heads/master@{#28734}
-
titzer authored
R=bmeurer@chromium.org,mstarzinger@chromium.org BUG= Review URL: https://codereview.chromium.org/1150083006 Cr-Commit-Position: refs/heads/master@{#28733}
-
ishell authored
This updates Dictionary classes hierarchy and introduces GlobalDictionary class but it is not used yet. Review URL: https://codereview.chromium.org/1163673003 Cr-Commit-Position: refs/heads/master@{#28732}
-