- 02 Aug, 2016 5 commits
-
-
franzih authored
BUG= Review-Url: https://codereview.chromium.org/2200713002 Cr-Commit-Position: refs/heads/master@{#38228}
-
franzih authored
Replace hydrogen stub for typeof with TurboFan stub. BUG=chromium:608675 Review-Url: https://codereview.chromium.org/2195863002 Cr-Commit-Position: refs/heads/master@{#38227}
-
machenbach authored
Revert of [builtins] implement Array.prototype.includes in TurboFan (patchset #20 id:380001 of https://codereview.chromium.org/2146293003/ ) Reason for revert: [Sheriff] Breaks: https://build.chromium.org/p/client.v8.ports/builders/V8%20Arm%20-%20builder/builds/2592 Original issue's description: > [builtins] implement Array.prototype.includes in TurboFan > > BUG=v8:5162 > R=bmeurer@chromium.org, ishell@chromium.org > > Committed: https://crrev.com/a488b5d8eb111a4883dc400bd826d079420edd68 > Cr-Commit-Position: refs/heads/master@{#38223} TBR=adamk@chromium.org,bmeurer@chromium.org,cbruni@chromium.org,danno@chromium.org,ishell@chromium.org,littledan@chromium.org,caitp@igalia.com # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=v8:5162 Review-Url: https://codereview.chromium.org/2202163002 Cr-Commit-Position: refs/heads/master@{#38226}
-
Michael Achenbach authored
Cr-Commit-Position: refs/heads/master@{#38225}
-
v8-autoroll authored
Rolling v8/build to a3a00fec14304015b590b283ba8ef6227aad4f53 Rolling v8/tools/mb to 65caad70eb36f22f3fcac6daa2f82365c0521657 TBR=machenbach@chromium.org,vogelheim@chromium.org,hablich@chromium.org Review-Url: https://codereview.chromium.org/2199203003 Cr-Commit-Position: refs/heads/master@{#38224}
-
- 01 Aug, 2016 35 commits
-
-
caitp authored
BUG=v8:5162 R=bmeurer@chromium.org, ishell@chromium.org Review-Url: https://codereview.chromium.org/2146293003 Cr-Commit-Position: refs/heads/master@{#38223}
-
alph authored
These are internal fields of the JSBoundFunction class. Review-Url: https://codereview.chromium.org/2193993002 Cr-Commit-Position: refs/heads/master@{#38222}
-
ahaas authored
The initialization of static variables that were used originally caused a data race because multiple threads tried to initialize the variables at the same time. The use of a LazyInstance guarantees that the variables get initialized exactly once. The same problem also existed in c-linkage.cc. There I fixed the problem by using a local variable instead of a static variable. BUG=v8:5242 R=titzer@chromium.org Review-Url: https://codereview.chromium.org/2202433003 Cr-Commit-Position: refs/heads/master@{#38221}
-
hpayer authored
BUG= Review-Url: https://codereview.chromium.org/2202633002 Cr-Commit-Position: refs/heads/master@{#38220}
-
klaasb authored
This will enable the interpreter to add a bytecode and use the stub. BUG=v8:4280 LOG=n Review-Url: https://codereview.chromium.org/2177273002 Cr-Commit-Position: refs/heads/master@{#38219}
-
hpayer authored
BUG= Review-Url: https://codereview.chromium.org/2185383002 Cr-Commit-Position: refs/heads/master@{#38218}
-
mattloring authored
Add deps file allowing libplatform.h to include v8-tracing.h. Additionally removes redundant include/ that was causing build errors for node-lkgr. BUG= Review-Url: https://codereview.chromium.org/2195403002 Cr-Commit-Position: refs/heads/master@{#38217}
-
jgruber authored
This is a temporary measure to ensure clusterfuzz crashes at two dedicated sites until the CallSite constructor is made inaccessible from JS. R=yangguo@chromium.org BUG= Review-Url: https://codereview.chromium.org/2196263002 Cr-Commit-Position: refs/heads/master@{#38216}
-
jgruber authored
BUG= Review-Url: https://codereview.chromium.org/2194883002 Cr-Commit-Position: refs/heads/master@{#38215}
-
machenbach authored
This introduces aliases for testing variants that can be combined with other variant names. E.g. --variants=dev,foo would run the three developer default variants and variant foo. We'll have three stages: "dev" for variants to be run by default on developer workstations, "more" for additional variants, executed on all bots, "extra" for additional variants executed on a subset of bots (e.g. not on very slow or otherwise resource-limited bots). BUG=v8:5238 NOTRY=true Review-Url: https://codereview.chromium.org/2196223002 Cr-Commit-Position: refs/heads/master@{#38214}
-
cbruni authored
BUG=chromium:630217 Review-Url: https://codereview.chromium.org/2196533003 Cr-Commit-Position: refs/heads/master@{#38213}
-
jgruber authored
BUG= Review-Url: https://codereview.chromium.org/2191293002 Cr-Commit-Position: refs/heads/master@{#38212}
-
ulan authored
introduced in https://crrev.com/72f884a19fa4434bba6fc0e013ec4ea0a2366893 The regression comes from adding the next weak field of AllocationSite as a hidden reference into the snapshot. Before 72f884 the reference was implicitly ignored because the body descriptor of AllocationSite did not include it. This patch explicitly skip the next weak field of AllocationSite. BUG=chromium:630027 Review-Url: https://codereview.chromium.org/2189643004 Cr-Commit-Position: refs/heads/master@{#38211}
-
verwaest authored
This makes the debug-only scope-name actually debug-only-allocated, replaces num_vars_ usages by variables_.occupancy, and shuffles fields around in the scope class for better packing. This reduces sizeof(i::Scope) from 360 to 328 bytes on x64. BUG=v8:5209 Review-Url: https://codereview.chromium.org/2201763004 Cr-Commit-Position: refs/heads/master@{#38210}
-
mstarzinger authored
This switches our inlining tests (i.e. cctest/test-run-inlining) to rely on global object instead of function context specialization, which is more in sync with what we are actually shipping. It will also allow us to test inlining with the BytecodeGraphBuilder without having to add support for function context specialization just for testing purposes. R=bmeurer@chromium.org TEST=cctest/test-run-inlining BUG=v8:5251 Review-Url: https://codereview.chromium.org/2200673002 Cr-Commit-Position: refs/heads/master@{#38209}
-
jgruber authored
BUG=632965 R=yangguo@chromium.org Review-Url: https://codereview.chromium.org/2199673002 Cr-Commit-Position: refs/heads/master@{#38208}
-
machenbach authored
BUG=chromium:631979 NOTRY=true Review-Url: https://codereview.chromium.org/2194823002 Cr-Commit-Position: refs/heads/master@{#38207}
-
machenbach authored
The flag was introduced for ignition development. It can only be used when running ignition tests in isolation on the bots. The bots only use ignition_turbo in isolation since a while and don't pass the --ignition flag anymore. BUG=v8:5238 Review-Url: https://codereview.chromium.org/2197123002 Cr-Commit-Position: refs/heads/master@{#38206}
-
mstarzinger authored
R=jarin@chromium.org Review-Url: https://codereview.chromium.org/2197163002 Cr-Commit-Position: refs/heads/master@{#38205}
-
jochen authored
Also remove unnecessary includes of scopeinfo.h all over the place R=marja@chromium.org TBR=verwaest@chromium.org BUG= Review-Url: https://codereview.chromium.org/2197973002 Cr-Commit-Position: refs/heads/master@{#38204}
-
jochen authored
BUG=v8:5215 R=vogelheim@chromium.org,marja@chromium.org Review-Url: https://codereview.chromium.org/2193813002 Cr-Commit-Position: refs/heads/master@{#38203}
-
machenbach authored
First step of deprecating the dedicated ignition alias. Next it will be changed on the bots to use the bot_default suite. Then we'll delete it. BUG=v8:5238 Review-Url: https://codereview.chromium.org/2194153002 Cr-Commit-Position: refs/heads/master@{#38202}
-
mstarzinger authored
This prevents the internal store-buffer.h header to be usable outisde of the "heap" directory. The logic inside that component is only useful within the GC and is now properly encapsulated. R=hpayer@chromium.org Review-Url: https://codereview.chromium.org/2194793005 Cr-Commit-Position: refs/heads/master@{#38201}
-
bmeurer authored
This adds initial support to inline a couple of the ArrayBuffer view accessors like %TypeArray%.prototype.length and. DataView.prototype.byteLength. R=epertoso@chromium.org Review-Url: https://codereview.chromium.org/2199753002 Cr-Commit-Position: refs/heads/master@{#38200}
-
cbruni authored
This CL fixes a long-standing bug with Object.keys where the enumerability check was omitted if the [ownKeys] trap is not present. The only distinction the KeyAccumulator needs is whether it collects keys for for-in (is_for_in_) or not. ForInFilter performs a separate step to filter out non-enumerable keys later-on while in all the other use-cases we have to filter keys. BUG=v8:1543, v8:5250 Review-Url: https://codereview.chromium.org/2176113009 Cr-Commit-Position: refs/heads/master@{#38199}
-
yangguo authored
BUG=chromium:580337 R=machenbach@chromium.org Review-Url: https://codereview.chromium.org/2189023002 Cr-Commit-Position: refs/heads/master@{#38198}
-
machenbach authored
BUG=chromium:631979 NOTRY=true Review-Url: https://codereview.chromium.org/2189343002 Cr-Commit-Position: refs/heads/master@{#38197}
-
verwaest authored
This is another step towards lazily allocating them in the block state. ClassLiteral should also have a lazy block-scope for the outermost scope, but currently that doesn't work due to the parameter initializer rewriter and minor implementation details in ignition and turbofan. BUG=v8:5209 Review-Url: https://codereview.chromium.org/2166843003 Cr-Commit-Position: refs/heads/master@{#38196}
-
hpayer authored
BUG=chromium:630969,chromium:630386 Review-Url: https://codereview.chromium.org/2186863005 Cr-Commit-Position: refs/heads/master@{#38195}
-
mstarzinger authored
This removes the frame state input representing the before-state from nodes having any shift operator. Any lowering that woult insert number conversions of the inputs has already been disabled when deoptimization is enabled, because the frame state layout is no longer known. R=epertoso@chromium.org BUG=v8:5021 Review-Url: https://codereview.chromium.org/2190743003 Cr-Commit-Position: refs/heads/master@{#38194}
-
verwaest authored
Rather than finalizing after rewriting do-expressions, we rewrite in the outer scope if the block scope was finalized. Rewriting do expressions cannot introduce any new nodes that requires the block to stay around, so finalizing before and after is equivalent. (Only a temporary is introduced which always ends up in a ClosureScope) BUG=v8:5209 R=rossberg@chromium.org, caitpotter88@gmail.com, adamk@chromium.org Review-Url: https://codereview.chromium.org/2167713004 Cr-Commit-Position: refs/heads/master@{#38193}
-
bmeurer authored
Allow inlining of getters and setters into TurboFan optimized code. This just adds the basic machinery required to essentially inline the setter and getter dispatch code for the (keyed) load/store ICs. There'll be follow up CLs to also actually inline some of the interesting accessor functions itself, like the byteLength and friends for the TypedArrays. R=jarin@chromium.org Review-Url: https://codereview.chromium.org/2198473002 Cr-Commit-Position: refs/heads/master@{#38192}
-
bmeurer authored
This introduces a bunch of new tests that test various aspects of accessor inlining in TurboFan (without the actual inlining), and does the appropriate fixes to the AstGraphBuilder. The actual inlining CL will land separately (so we don't need to revert the tests and fixes if the accessor CL has to be reverted). R=jarin@chromium.org Review-Url: https://codereview.chromium.org/2197913002 Cr-Commit-Position: refs/heads/master@{#38191}
-
yangguo authored
BUG=chromium:580337 Committed: https://crrev.com/b8229434c9fd3bfef81fb77d88779d0e78d06662 Review-Url: https://codereview.chromium.org/2190833004 Cr-Original-Commit-Position: refs/heads/master@{#38135} Cr-Commit-Position: refs/heads/master@{#38190}
-
v8-autoroll authored
Rolling v8/build to 94ae8edf4860b0dfa8ac200d36bcbf11bdd72763 Rolling v8/tools/mb to d1d562a498b7b48a283d168df902007f33ac1413 TBR=machenbach@chromium.org,vogelheim@chromium.org,hablich@chromium.org Review-Url: https://codereview.chromium.org/2194113002 Cr-Commit-Position: refs/heads/master@{#38189}
-