- 15 Sep, 2016 6 commits
-
-
v8-autoroll authored
Rolling v8/build to a34a5233778556481dfa869bff735fad2157f196 Rolling v8/third_party/WebKit/Source/platform/inspector_protocol to e240fdcdb5880deb48156dbb9ccee0c28664cf88 Rolling v8/third_party/instrumented_libraries to 45f5814b1543e41ea0be54c771e3840ea52cca4a TBR=machenbach@chromium.org,vogelheim@chromium.org,hablich@chromium.org Review-Url: https://codereview.chromium.org/2347533002 Cr-Commit-Position: refs/heads/master@{#39436}
-
littledan authored
This flag has been flipped off since 52, so it is due for removal. R=adamk@chromium.org,caitp@igalia.com BUG=v8:3785 CQ_INCLUDE_TRYBOTS=master.tryserver.v8:v8_linux_noi18n_rel_ng Review-Url: https://codereview.chromium.org/2268633002 Cr-Commit-Position: refs/heads/master@{#39435}
-
neis authored
In case of duplicate exports, always report the error for the very last one. (Fixed a bug.) BUG=v8:5358,v8:1569 Review-Url: https://codereview.chromium.org/2340953002 Cr-Commit-Position: refs/heads/master@{#39434}
-
littledan authored
Handle the "synchronous case" by marking try/catch blocks introduced for async functions as ASYNC_AWAIT and traversing up the stack, finding successive Promises and returning caught if any of them are predicted to be caught. BUG=v8:5167 Review-Url: https://codereview.chromium.org/2325813002 Cr-Commit-Position: refs/heads/master@{#39433}
-
hablich authored
Revert of [inspector] fixed all shorten-64-to-32 warnings (patchset #4 id:80001 of https://codereview.chromium.org/2332163002/ ) Reason for revert: Blocking V8 roll: https://codereview.chromium.org/2347463002/ See https://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_ng/builds/293368 for compile error. Original issue's description: > [inspector] fixed all shorten-64-to-32 warnings > > BUG=chromium:635948 > R=dgozman@chromium.org,alph@chromium.org > > Committed: https://crrev.com/3d10918d2e1c57d72531c55a956262f5a72fceaa > Cr-Commit-Position: refs/heads/master@{#39426} TBR=jochen@chromium.org,alph@chromium.org,dgozman@chromium.org,kozyatinskiy@chromium.org # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=chromium:635948 Review-Url: https://codereview.chromium.org/2339173004 Cr-Commit-Position: refs/heads/master@{#39432}
-
hablich authored
Revert of [inspector] removed ArrayLengthHelperFunction specialization for empty array (patchset #2 id:20001 of https://codereview.chromium.org/2340623002/ ) Reason for revert: needed to revert https://codereview.chromium.org/2339173004/ Original issue's description: > [inspector] removed ArrayLengthHelperFunction specialization for empty array > > There is no zero length array usage in inspector codebase. We can safely remove template specialization. > > BUG=chromium:635948 > R=dgozman@chromium.org,alph@chromium.org > > Committed: https://crrev.com/b249ffc1915d8c35ff66eae3d51d3878d588dfbf > Cr-Commit-Position: refs/heads/master@{#39428} TBR=jochen@chromium.org,alph@chromium.org,dgozman@chromium.org,kozyatinskiy@chromium.org # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=chromium:635948 Review-Url: https://codereview.chromium.org/2334163006 Cr-Commit-Position: refs/heads/master@{#39431}
-
- 14 Sep, 2016 34 commits
-
-
hablich authored
Revert of [modules] Make duplicate export error deterministic. (patchset #2 id:20001 of https://codereview.chromium.org/2331003002/ ) Reason for revert: Seems to break one of our arm64 bots: https://chromegw.corp.google.com/i/client.v8.ports/builders/V8%20Linux%20-%20arm64%20-%20sim%20-%20nosnap%20-%20debug/builds/2303 Original issue's description: > [modules] Make duplicate export error deterministic. > > In case of duplicate exports, always report the error for the very last > one. > > R=adamk@chromium.org > BUG=v8:5358,v8:1569 > > Committed: https://crrev.com/da1f911c4269048d24a3442791b18523455f3b24 > Cr-Commit-Position: refs/heads/master@{#39424} TBR=adamk@chromium.org,neis@chromium.org # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=v8:5358,v8:1569 Review-Url: https://codereview.chromium.org/2340903002 Cr-Commit-Position: refs/heads/master@{#39430}
-
jbroman authored
Similar to json-parser. BUG=chromium:148757 Review-Url: https://codereview.chromium.org/2334353002 Cr-Commit-Position: refs/heads/master@{#39429}
-
kozyatinskiy authored
There is no zero length array usage in inspector codebase. We can safely remove template specialization. BUG=chromium:635948 R=dgozman@chromium.org,alph@chromium.org Review-Url: https://codereview.chromium.org/2340623002 Cr-Commit-Position: refs/heads/master@{#39428}
-
jochen authored
BUG=chromium:646794 R=machenbach@chromium.org Review-Url: https://codereview.chromium.org/2339933002 Cr-Commit-Position: refs/heads/master@{#39427}
-
kozyatinskiy authored
BUG=chromium:635948 R=dgozman@chromium.org,alph@chromium.org Review-Url: https://codereview.chromium.org/2332163002 Cr-Commit-Position: refs/heads/master@{#39426}
-
bjaideep authored
Port 3ccedd5d R=mvstanton@chromium.org, joransiu@ca.ibm.com, jyan@ca.ibm.com, michael_dawson@ca.ibm.com, mbrandy@us.ibm.com BUG= LOG=N Review-Url: https://codereview.chromium.org/2330353005 Cr-Commit-Position: refs/heads/master@{#39425}
-
neis authored
In case of duplicate exports, always report the error for the very last one. R=adamk@chromium.org BUG=v8:5358,v8:1569 Review-Url: https://codereview.chromium.org/2331003002 Cr-Commit-Position: refs/heads/master@{#39424}
-
jarin authored
Review-Url: https://codereview.chromium.org/2337283002 Cr-Commit-Position: refs/heads/master@{#39423}
-
jbroman authored
This exposes an interface for the embedder to provide a delegate which can serialize or deserialize embedder-specific objects, like Blink's DOM wrappers. BUG=chromium:148757 Review-Url: https://codereview.chromium.org/2327653002 Cr-Commit-Position: refs/heads/master@{#39422}
-
ahaas authored
TEST=unittest WasmModuleVerifyTest.Global_invalid_type2 BUG=chromium:645443 R=titzer@chromium.org Review-Url: https://codereview.chromium.org/2342593002 Cr-Commit-Position: refs/heads/master@{#39421}
-
bmeurer authored
Add a dedicated simplified operator to inline the general case for the ToBoolean conversion. In a follow up CL we will also use the ToBoolean hints gathered by the baseline compiler. CQ_INCLUDE_TRYBOTS=master.tryserver.v8:v8_linux_arm64_gc_stress_dbg R=jarin@chromium.org BUG=v8:5267 Committed: https://crrev.com/8c50b51ab3d21efcd2f6900d83962159f21e1590 Review-Url: https://codereview.chromium.org/2167593002 Cr-Original-Commit-Position: refs/heads/master@{#37882} Cr-Commit-Position: refs/heads/master@{#39420}
-
ahaas authored
R=titzer@chromium.org BUG=chromium:646258 Review-Url: https://codereview.chromium.org/2341673002 Cr-Commit-Position: refs/heads/master@{#39419}
-
mvstanton authored
BUG= Review-Url: https://codereview.chromium.org/2342533002 Cr-Commit-Position: refs/heads/master@{#39418}
-
jgruber authored
RegExpSubclassExecJS and RegExpExecJS only differed in the additional TO_BOOLEAN on global and sticky flags and the useless (i < 0) check. R=littledan@chromium.org BUG=v8:5339 Review-Url: https://codereview.chromium.org/2337923002 Cr-Commit-Position: refs/heads/master@{#39417}
-
machenbach authored
BUG=chromium:474921 NOTRY=true Review-Url: https://codereview.chromium.org/2334003005 Cr-Commit-Position: refs/heads/master@{#39416}
-
Ilija.Pavlovic authored
Implementation MADD.S. MSUB.fmt, MADDF.fmt, MSUBF.fmt and corresponding tests for assembler and disassembler. TEST=cctest/test-assembler-mips[64], cctest/test-disasm-mips[64] BUG= Review-Url: https://codereview.chromium.org/2313623002 Cr-Commit-Position: refs/heads/master@{#39415}
-
machenbach authored
This restores the original gyp behavior. Blocks: https://codereview.chromium.org/2334003005 BUG=chromium:474921 NOTRY=true Review-Url: https://codereview.chromium.org/2344493002 Cr-Commit-Position: refs/heads/master@{#39414}
-
ahaas authored
This CL adds fuzzers for the wasm module sections 'types', 'names', 'globals', 'imports', 'function signatures', 'memory', and 'data', one fuzzer per section. No fuzzers are added for the other sections because either there already exists a fuzzer (e.g. wasm-code), or there exist inter-section dependencies. To avoid introducing a bunch executables which would make compilation with make slow, I introduce a single executable 'v8_simple_wasm_section_fuzzer' which calls the fuzzers mentioned above. This executable is run by the trybots and ensures that the fuzzers actually compile. For debugging I introduce commandline parameters which allow to execute the specific fuzzers from 'v8_simple_wasm_section_fuzzer'. R=titzer@chromium.org, jochen@chromium.org, mstarzinger@chromium.org Review-Url: https://codereview.chromium.org/2336603002 Cr-Commit-Position: refs/heads/master@{#39413}
-
jochen authored
This is required to be able to deserialize the scope chain from the scope info alone. BUG=v8:5215 R=marja@chromium.org,jgruber@chromium.org Review-Url: https://codereview.chromium.org/2331323006 Cr-Commit-Position: refs/heads/master@{#39412}
-
ahaas authored
The wasm-module-runner is used both in cctests and in fuzzers. As discussed offline, it is weird to include cctest header files in fuzzers, so I introduce a new test/common directory which contains the common files. R=titzer@chromium.org, jochen@chromium.org Review-Url: https://codereview.chromium.org/2335193002 Cr-Commit-Position: refs/heads/master@{#39411}
-
bmeurer authored
Add a notion of "invocation count" to the baseline compilers, which increment a special slot in the TypeFeedbackVector for each invocation of a given function (the optimized code doesn't currently collect this information). Use this invocation count to relativize the call counts on the call sites within the function, so that the inlining heuristic has a view of relative importance of a call site rather than some absolute numbers with unclear meaning for the current function. Also apply the call site frequency as a factor to all frequencies in the inlinee by passing this to the graph builders so that the importance of a call site in an inlinee is relative to the topmost optimized function. Note that all functions that neither have literals nor need type feedback slots will share a single invocation count cell in the canonical empty type feedback vector, so their invocation count is meaningless, but that doesn't matter since we only use the invocation count to relativize call counts within the function, which we only have if we have at least one type feedback vector (the CallIC slot). See the design document for additional details on this change: https://docs.google.com/document/d/1VoYBhpDhJC4VlqMXCKvae-8IGuheBGxy32EOgC2LnT8 BUG=v8:5267,v8:5372 R=mvstanton@chromium.org,rmcilroy@chromium.org,mstarzinger@chromium.org Review-Url: https://codereview.chromium.org/2337123003 Cr-Commit-Position: refs/heads/master@{#39410}
-
georgia.kouveli authored
BUG= Review-Url: https://codereview.chromium.org/2337953003 Cr-Commit-Position: refs/heads/master@{#39409}
-
Alexander.Gilday2 authored
All events recorded are shown in separate columns simulatneously, using rectangles with heatmap-style colouring. Hovering over the shapes gives the event name, count, and percentage. BUG= Review-Url: https://codereview.chromium.org/2228553004 Cr-Commit-Position: refs/heads/master@{#39408}
-
ishell authored
This CL adds CSA::Retain() operation that ensures that the value is kept alive even during GC. BUG=v8:5269 Review-Url: https://codereview.chromium.org/2330063002 Cr-Commit-Position: refs/heads/master@{#39407}
-
ahaas authored
R=titzer@chromium.org Review-Url: https://codereview.chromium.org/2341653002 Cr-Commit-Position: refs/heads/master@{#39406}
-
ahaas authored
BUG=chromium:646564 R=titzer@chromium.org Review-Url: https://codereview.chromium.org/2336363003 Cr-Commit-Position: refs/heads/master@{#39405}
-
mstarzinger authored
This removes some leftover code which avoided adding stack checks to stubs being compiled via the normal JavaScript pipeline, which we no longer do. R=bmeurer@chromium.org Review-Url: https://codereview.chromium.org/2333973003 Cr-Commit-Position: refs/heads/master@{#39404}
-
jgruber authored
R=littledan@chromium.org BUG=v8:5339 Review-Url: https://codereview.chromium.org/2337763003 Cr-Commit-Position: refs/heads/master@{#39403}
-
jgruber authored
This implements https://github.com/tc39/ecma262/pull/627/. BUG=v8:5360 Review-Url: https://codereview.chromium.org/2339443002 Cr-Commit-Position: refs/heads/master@{#39402}
-
jgruber authored
R=littledan@chromium.org BUG=v8:5339 Review-Url: https://codereview.chromium.org/2330413002 Cr-Commit-Position: refs/heads/master@{#39401}
-
bmeurer authored
Extract the call counts from the type feedback vector during graph building (either via the AstGraphBuilder or the BytecodeGraphBuilder), and put them onto the JSCallFunction and JSCallConstruct operators, so that they work even across inlinine through .apply and .call (which was previously hacked by creating a temporary type feedback vector for those). The next logic step will be to make those call counts into real relative call frequencies (also during graph building), so that we can make inlining decisions that make sense for the function being optimized (where absolute values are misleading). R=jarin@chromium.org BUG=v8:5267,v8:5372 Review-Url: https://codereview.chromium.org/2330883002 Cr-Commit-Position: refs/heads/master@{#39400}
-
v8-autoroll authored
Rolling v8/build to 4e62debd8bf6e75229861c7b8b352953f8bfec2e Rolling v8/buildtools to b97d6c93a3e805c9ba5356dca872f1801639fbc5 Rolling v8/third_party/WebKit/Source/platform/inspector_protocol to f010dd827c6d3a8ec2f355168fc4342ba39a78f8 Rolling v8/third_party/icu to b0bd3ee50bc2e768d7a17cbc60d87f517f024dbe Rolling v8/tools/clang to 23638ab6411fb9e25ea7f1837835b67a793b5499 TBR=machenbach@chromium.org,vogelheim@chromium.org,hablich@chromium.org Review-Url: https://codereview.chromium.org/2343533002 Cr-Commit-Position: refs/heads/master@{#39399}
-
mvstanton authored
BUG= Review-Url: https://codereview.chromium.org/2333243004 Cr-Commit-Position: refs/heads/master@{#39398}
-
hablich authored
Revert of [heap] Enable black allocation when finalizing incremental marking. (patchset #6 id:100001 of https://codereview.chromium.org/2239733002/ ) Reason for revert: Suspecting this is a roll blocker: https://codereview.chromium.org/2332343002/ # # Fatal error in ../../v8/src/heap/spaces.h, line 1618 # Check failed: capacity_ >= bytes (25429824 vs. 18446744073709551168). # Original issue's description: > [heap] Enable black allocation when finalizing incremental marking. > > BUG=chromium:630386 > > Committed: https://crrev.com/1ccc742dc326b063999670421e1da0ea124874c0 > Cr-Commit-Position: refs/heads/master@{#39382} TBR=mlippautz@chromium.org,hpayer@chromium.org # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=chromium:630386 Review-Url: https://codereview.chromium.org/2340463005 Cr-Commit-Position: refs/heads/master@{#39397}
-