- 21 Nov, 2018 31 commits
-
-
Daniel Clifford authored
Also add the first intrinsic and usage of it: %RawCast Bug: v8:7793 Change-Id: Id1e3288e8bab6adb510731076a39590e8fd156be Reviewed-on: https://chromium-review.googlesource.com/c/1344152 Commit-Queue: Daniel Clifford <danno@chromium.org> Reviewed-by:
Tobias Tebbi <tebbi@chromium.org> Cr-Commit-Position: refs/heads/master@{#57692}
-
Tobias Tebbi authored
Otherwise, it will always replace LF with CRLF, which is not what you want if you follow the Chromium instructions for Windows, that is, configure git with core.autocrlf = false. Change-Id: I30fcfc471cde79d5c80d05ce582a8507cf5810b5 Reviewed-on: https://chromium-review.googlesource.com/c/1345150Reviewed-by:
Daniel Clifford <danno@chromium.org> Commit-Queue: Tobias Tebbi <tebbi@chromium.org> Cr-Commit-Position: refs/heads/master@{#57691}
-
Sergiy Byelozyorov authored
TBR=sergiyb@chromium.org Bug: chromium:903688 Change-Id: I72b57d45286ae28c44bad0edffeb56e9142767d1 Reviewed-on: https://chromium-review.googlesource.com/c/1346494Reviewed-by:
Sergiy Byelozyorov <sergiyb@chromium.org> Commit-Queue: Sergiy Byelozyorov <sergiyb@chromium.org> Cr-Commit-Position: refs/heads/master@{#57690}
-
Yang Guo authored
TBR=aseemgarg@chromium.org Change-Id: Iabb0a1d5ac065f97baf9b44815f0c5df37b426b5 Reviewed-on: https://chromium-review.googlesource.com/c/1346493Reviewed-by:
Yang Guo <yangguo@chromium.org> Commit-Queue: Yang Guo <yangguo@chromium.org> Cr-Commit-Position: refs/heads/master@{#57689}
-
Sigurd Schneider authored
Load elimination is running together with to dead code elimination, the latter of which might eliminate allocations (in particular FinishRegion nodes). These are treated as alias nodes by load elimination, and load elimination does not immediatelly learn that a node has been disconnected. This causes load elimination to access the inputs of dead code eliminated nodes while resolving renames, which causes nullptr dereferences. This CL modifies load elimination to not resolve to a nullptr alias but simply stop before that. Change-Id: If4cef061c7c0e25f353727c9e27f790439b0beb5 Bug: chromium:906406 Reviewed-on: https://chromium-review.googlesource.com/c/1346491 Commit-Queue: Sigurd Schneider <sigurds@chromium.org> Reviewed-by:
Jaroslav Sevcik <jarin@chromium.org> Cr-Commit-Position: refs/heads/master@{#57688}
-
Stephan Herhut authored
Adds support to render the live ranges as ascii diagram. This is similar in nature to what the c1visualizer would produce. Also, print the visualization when tracing the register allocator. Change-Id: Ib6a43f67ba356e9a80cacaddfdbd6d589c685483 Reviewed-on: https://chromium-review.googlesource.com/c/1346114Reviewed-by:
Sigurd Schneider <sigurds@chromium.org> Commit-Queue: Stephan Herhut <herhut@chromium.org> Cr-Commit-Position: refs/heads/master@{#57687}
-
Hannes Payer authored
Bug: chromium:852420 Change-Id: I54d1708991468bb062d9e7b246e724594f252e5e Reviewed-on: https://chromium-review.googlesource.com/c/1346369 Commit-Queue: Hannes Payer <hpayer@chromium.org> Reviewed-by:
-
Sergiy Byelozyorov authored
TBR=machenbach@chromium.org No-Try: true Bug: chromium:616879 Change-Id: I7a567060d4bbfdd5cbd8cc92a828b051ce8ddb8c Reviewed-on: https://chromium-review.googlesource.com/c/1346332 Commit-Queue: Sergiy Byelozyorov <sergiyb@chromium.org> Reviewed-by:
-
Toon Verwaest authored
This moves is_async and parenthesized_formals tracking out of the ParseAssignmentExpression header. Instead we set flags on the parser for the next arrow function: next_arrow_function_kind_ that defaults to kArrowFunction and next_arrow_formals_parenthesized_ which defaults to false. If we end up recognizing an async arrow head, we update next_arrow_function_kind_ to kAsyncArrowFunction. If we detect (...) followed by => or async(...) followed by => we set next_arrow_formals_parenthesized_ to true. When we start parsing the arrow continuation we consume the values and reset them to default for the next arrow function. Change-Id: Ia22c0068fea0f175fafc568b7339813fd10c3b5a Reviewed-on: https://chromium-review.googlesource.com/c/1345996 Commit-Queue: Toon Verwaest <verwaest@chromium.org> Reviewed-by:
Igor Sheludko <ishell@chromium.org> Cr-Commit-Position: refs/heads/master@{#57684}
-
Toon Verwaest authored
It seems that allocating smaller pages is actually quite a bit faster than larger pages, probably because they can be cached by malloc. Let's see what the bots say. In a follow-up I'll check whether the segment-pool is actually beneficial or whether we should just remove it. This also drops SegmentSize::kLarge as a way to make compilation deterministic. Turns out that by now we need >8mb anyway, and the previous 1mb wasn't enough. At the same time the compiler was fixed to not rely on virtual addresses of zone objects anymore, and there's a bot checking whether the snapshot is determistic. Change-Id: I38cbb0d209d68b3671fd38763b42714811f4223e Reviewed-on: https://chromium-review.googlesource.com/c/1346370Reviewed-by:
Ulan Degenbaev <ulan@chromium.org> Commit-Queue: Toon Verwaest <verwaest@chromium.org> Cr-Commit-Position: refs/heads/master@{#57683}
-
Toon Verwaest authored
Use the parser's IsValidReferenceExpression as a likely-succeeding precheck. Slightly optimizes IsEvalOrArguments in the preparser and IsIdentifier for the parser (we now have FailureExpression everywhere); and replaces IsObjectLiteral||IsArrayLiteral by IsValidPattern. Change-Id: I7e9684485c0ce454e640800566eb4b0a24c6bfc8 Reviewed-on: https://chromium-review.googlesource.com/c/1345995 Commit-Queue: Toon Verwaest <verwaest@chromium.org> Reviewed-by:
-
tzik authored
This is an implementation of https://bit.ly/v8-faster-microtask-queues step 2. This CL overhauls MicrotaskQueue class, the previous one is on V8 heap, and the new one is on C++ heap. Benchmark: This CL improves a benchmark score around promise by 5~23%. https://github.com/v8/promise-performance-tests https://docs.google.com/spreadsheets/d/1HtwZGzUAGJYg87VmYhV9hLdvfddlCtC6Oz0iOj-WwQA/edit#gid=1952666737 Bug: chromium:887920, v8:7253 Change-Id: I1f26e02c45ae60ae39d1ccc168daa98bca4663d9 Reviewed-on: https://chromium-review.googlesource.com/c/1290751 Commit-Queue: Taiju Tsuiki <tzik@chromium.org> Reviewed-by:
Adam Klein <adamk@chromium.org> Reviewed-by:
Benedikt Meurer <bmeurer@chromium.org> Reviewed-by:
-
Sergiy Byelozyorov authored
R=efoo@chromium.org, machenbach@chromium.org Bug: chromium:868202 Change-Id: I5c0860894c96a8ce14e74cb048f786908db88269 Reviewed-on: https://chromium-review.googlesource.com/c/1344150 Commit-Queue: Sergiy Byelozyorov <sergiyb@chromium.org> Reviewed-by:
Michael Achenbach <machenbach@chromium.org> Cr-Commit-Position: refs/heads/master@{#57680}
-
Igor Sheludko authored
an abstraction that hides the details about how embedder data fields are encoded in EmbedderDataArrays and JSObjects. Bug: v8:7703 Change-Id: Ic9f6d9511bec557c6671aa6488b9545e353f968c Reviewed-on: https://chromium-review.googlesource.com/c/1344155 Commit-Queue: Igor Sheludko <ishell@chromium.org> Reviewed-by:
-
Clemens Hammacher authored
This reverts commit e8a22a70. Reason for revert: Depends on another reverted CL: https://crrev.com/c/1337585 Original change's description: > [iwyu] Add missing includes > > This fixes a presubmit error: > https://ci.chromium.org/p/v8/builders/luci.v8.ci/V8%20Presubmit/454 > > It also adds another missing include. > > R=machenbach@chromium.org > CC=caitp@igalia.com > > Bug: v8:7490 > Change-Id: I915a5bf431073410a67ff08471fd5e6974e31aa1 > No-Try: true > No-Tree-Checks: true > Reviewed-on: https://chromium-review.googlesource.com/c/1346111 > Reviewed-by: Michael Achenbach <machenbach@chromium.org> > Commit-Queue: Clemens Hammacher <clemensh@chromium.org> > Cr-Commit-Position: refs/heads/master@{#57671} TBR=machenbach@chromium.org,clemensh@chromium.org Change-Id: Id01d5ff327bc50f8d21a4092ba881f5d5e982a92 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: v8:7490 Reviewed-on: https://chromium-review.googlesource.com/c/1346117Reviewed-by:
Clemens Hammacher <clemensh@chromium.org> Commit-Queue: Clemens Hammacher <clemensh@chromium.org> Cr-Commit-Position: refs/heads/master@{#57678}
-
Clemens Hammacher authored
This reverts commit a5336471. Reason for revert: Fails nosnap debug tests: https://ci.chromium.org/p/v8/builders/luci.v8.ci/V8%20Linux%20-%20nosnap%20-%20debug/21838 Original change's description: > [builtins] Implement Object.fromEntries > > Adds the Object.fromEntries() method behind > --harmony-object-from-entries. > > > Includes an initial implementation of the new experimental builtin > Object.fromEntries implemented by Daniel Clifford, and > has been modified by Caitlin Potter to support a fast case to skip > the iterator protocol when it can be done unobservably in common cases. > > There are some incidental changes: A number of CSA macros have been > updated to use TNodes, and some Context arguments have been > re-arranged to be implicit in Torque. > > > There are also a number of mjsunit tests written mirroring and > expanding on the test262 tests. > > BUG=v8:8021 > > Change-Id: I1c12bee8a2f98c6297b77d5d723910a5e3b630cc > Co-authored-by: Daniel Clifford <danno@chromium.org> > Co-authored-by: Caitlin Potter <caitp@igalia.com> > Reviewed-on: https://chromium-review.googlesource.com/c/1337585 > Commit-Queue: Daniel Clifford <danno@chromium.org> > Reviewed-by: Daniel Clifford <danno@chromium.org> > Reviewed-by: Tobias Tebbi <tebbi@chromium.org> > Cr-Commit-Position: refs/heads/master@{#57667} TBR=danno@chromium.org,caitp@igalia.com,tebbi@chromium.org Change-Id: Id0cd8b16131f151a42dffbaca7e59ab17c68ab23 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: v8:8021 Reviewed-on: https://chromium-review.googlesource.com/c/1346116Reviewed-by:
-
Georg Neis authored
R=jarin@chromium.org Bug: chromium:906567 Change-Id: I280569e09529457814ebc1cb8f299a80aa40a6c4 Reviewed-on: https://chromium-review.googlesource.com/c/1345992Reviewed-by:
-
Camillo Bruni authored
- Show spinner while parsing input - Show color boxes next transition types - Support drag-n-drop of v8.log files No-Try: true Change-Id: I360d62960e9ec05fbab388ee2ca55baf4e2c6b78 Reviewed-on: https://chromium-review.googlesource.com/c/1345151Reviewed-by:
Michael Lippautz <mlippautz@chromium.org> Commit-Queue: Camillo Bruni <cbruni@chromium.org> Cr-Commit-Position: refs/heads/master@{#57675}
-
Camillo Bruni authored
No-Try: true Change-Id: I078279bef7141c0255025efebfd3a39a57fc3bf7 Reviewed-on: https://chromium-review.googlesource.com/c/1345149Reviewed-by:
-
Ulan Degenbaev authored
This extracts the parts of the TypedSlotSet that are used only sequentially into a separate class called TypedSlots. The new class will be used in the concurrent marker to keep track of typed slots locally and then to merge them to the main remembered set during finalization of marking. The patch also cleans up atomics in the Iterate and ClearInvalidSlots methods that can run concurrently to each other. Bug:v8:8459 Change-Id: Id7a63041f7b99218381e5e9e1999210cab9c4369 Reviewed-on: https://chromium-review.googlesource.com/c/1340247 Commit-Queue: Ulan Degenbaev <ulan@chromium.org> Reviewed-by:
Hannes Payer <hpayer@chromium.org> Reviewed-by:
-
Yang Guo authored
R=leszeks@chromium.org Change-Id: Iebf284d3ac041eecb8cfb832c7331febb1ccf089 Reviewed-on: https://chromium-review.googlesource.com/c/1346069Reviewed-by:
Leszek Swirski <leszeks@chromium.org> Commit-Queue: Yang Guo <yangguo@chromium.org> Cr-Commit-Position: refs/heads/master@{#57672}
-
Clemens Hammacher authored
This fixes a presubmit error: https://ci.chromium.org/p/v8/builders/luci.v8.ci/V8%20Presubmit/454 It also adds another missing include. R=machenbach@chromium.org CC=caitp@igalia.com Bug: v8:7490 Change-Id: I915a5bf431073410a67ff08471fd5e6974e31aa1 No-Try: true No-Tree-Checks: true Reviewed-on: https://chromium-review.googlesource.com/c/1346111Reviewed-by:
-
Sigurd Schneider authored
Instead of attaching an event handler to every line in the code view, attach it only to the container and find the lineNumber based on the event target element. Notry: true Change-Id: I1920f7a200cf2f5ffaf259c0aaa04d6fb6698d2d Bug: v8:7327 Reviewed-on: https://chromium-review.googlesource.com/c/1346110 Commit-Queue: Sigurd Schneider <sigurds@chromium.org> Reviewed-by:
Georg Neis <neis@chromium.org> Cr-Commit-Position: refs/heads/master@{#57670}
-
Igor Sheludko authored
and prepare for adding an EmbedderDataSlot which will occupy two tagged slots. Bug: v8:8477, v8:8238, v8:7703 Change-Id: I8aa0a0c9b64835ad7f847033bcc55dd3b3c43563 Reviewed-on: https://chromium-review.googlesource.com/c/1344153 Commit-Queue: Igor Sheludko <ishell@chromium.org> Reviewed-by:
Toon Verwaest <verwaest@chromium.org> Cr-Commit-Position: refs/heads/master@{#57669}
-
Clemens Hammacher authored
Register names are static, so we do not need to access them via RegisterConfiguration. This saves a lot of RegisterConfiguration object creations. R=mstarzinger@chromium.org Bug: v8:8238 Change-Id: I295ad4d4b13fe948c70490687b7e3e9b48e70af9 Reviewed-on: https://chromium-review.googlesource.com/c/1342517Reviewed-by:
Michael Starzinger <mstarzinger@chromium.org> Commit-Queue: Clemens Hammacher <clemensh@chromium.org> Cr-Commit-Position: refs/heads/master@{#57668}
-
Caitlin Potter authored
Adds the Object.fromEntries() method behind --harmony-object-from-entries. Includes an initial implementation of the new experimental builtin Object.fromEntries implemented by Daniel Clifford, and has been modified by Caitlin Potter to support a fast case to skip the iterator protocol when it can be done unobservably in common cases. There are some incidental changes: A number of CSA macros have been updated to use TNodes, and some Context arguments have been re-arranged to be implicit in Torque. There are also a number of mjsunit tests written mirroring and expanding on the test262 tests. BUG=v8:8021 Change-Id: I1c12bee8a2f98c6297b77d5d723910a5e3b630cc Co-authored-by:
Caitlin Potter <caitp@igalia.com> Reviewed-on: https://chromium-review.googlesource.com/c/1337585 Commit-Queue: Daniel Clifford <danno@chromium.org> Reviewed-by:
-
Jakob Gruber authored
Verification has served its purpose now that kRootRegister is fully supported on ia32. Bug: v8:6666 Change-Id: I037617bf900f161c221228107c9f1352d4815d87 Reviewed-on: https://chromium-review.googlesource.com/c/1296130Reviewed-by:
-
Toon Verwaest authored
This reverts commit 5616f91a. Reason for revert: Tanks performance on codeload/ia32 by >1% Original change's description: > [parser] Use memcmp for keyword checks > > Small readability increase for the keyword check magic, using memcmp > instead of a chain of raw comparisons. Could allow better codegen for > memcmp-aware compilers, though in practice seems to have little effect > on generated code. > > Change-Id: I91020fe67cebc9270c61c4c678e15217e436afff > Reviewed-on: https://chromium-review.googlesource.com/c/1340291 > Reviewed-by: Toon Verwaest <verwaest@chromium.org> > Commit-Queue: Leszek Swirski <leszeks@chromium.org> > Cr-Commit-Position: refs/heads/master@{#57595} TBR=leszeks@chromium.org,verwaest@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Change-Id: Ifd4070efb3e692eb70e5d8b3adb04d7d7e45ef0e Reviewed-on: https://chromium-review.googlesource.com/c/1345994Reviewed-by:
-
Benedikt Meurer authored
This is a reland of 585b4eef without any changes. Original change's description: > [turbofan] Improve NumberMultiply typing rule. > > The NumberMultiply typing rule gave up in the presence of NaN inputs, > but we can still infer useful ranges here and just union the result > of that with the NaN propagation (similar for MinusZero propagation). > This way we can still makes sense of these ranges at the uses. > > Bug: v8:8015 > Change-Id: Ic4c5e8edc6c68776ff3baca9628ad7de0f8e2a92 > Reviewed-on: https://chromium-review.googlesource.com/c/1261143 > Commit-Queue: Benedikt Meurer <bmeurer@chromium.org> > Reviewed-by: Sigurd Schneider <sigurds@chromium.org> > Cr-Commit-Position: refs/heads/master@{#56539} Tbr: bmeurer@chromium.org Bug: v8:8015 Change-Id: I32e5c2f439a1186891ca3393ee53a2a766585839 Reviewed-on: https://chromium-review.googlesource.com/c/1345993Reviewed-by:
-
Michael Achenbach authored
Rolling v8/build: https://chromium.googlesource.com/chromium/src/build/+log/1f137f3..a2bfd53 Rolling v8/buildtools: https://chromium.googlesource.com/chromium/buildtools/+log/13a00f1..da9b294 Rolling v8/test/wasm-js/data: https://chromium.googlesource.com/external/github.com/WebAssembly/spec/+log/5aaea96..e822213 Rolling v8/third_party/catapult: https://chromium.googlesource.com/catapult/+log/1e54003..d115075 Rolling v8/third_party/depot_tools: https://chromium.googlesource.com/chromium/tools/depot_tools/+log/7da982a..e47ac15 Rolling v8/tools/swarming_client: https://chromium.googlesource.com/infra/luci/client-py/+log/7f463e6..b6e9e23 TBR=machenbach@chromium.org,hablich@chromium.org,sergiyb@chromium.org Change-Id: Icf622c3e49371ba965f08e15d0177697f04c55e5 Reviewed-on: https://chromium-review.googlesource.com/c/1344806 Commit-Queue: Michael Achenbach <machenbach@chromium.org> Reviewed-by:
v8-ci-autoroll-builder <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#57663}
-
Yuki Shiino authored
When ASAN is enabled, the previous implementation of Isolate::GetIncumbentContext didn't work well due to mixture of fake and real stack frames. This patch converts an address in the fake stack frame to an address in the real stack frame so that we can compare two addresses. Bug: chromium:888867, chromium:866610 Change-Id: Iccf570b8555f2fbdc737b12894a2784ffdb31602 Reviewed-on: https://chromium-review.googlesource.com/c/1343709Reviewed-by:
Camillo Bruni <cbruni@chromium.org> Commit-Queue: Yuki Shiino <yukishiino@chromium.org> Cr-Commit-Position: refs/heads/master@{#57662}
-
- 20 Nov, 2018 9 commits
-
-
Igor Sheludko authored
This reverts commit a6e3cdd9. Reason for revert: https://ci.chromium.org/p/v8/builders/luci.v8.ci/V8%20Android%20GN%20%28dbg%29/14259 Original change's description: > [cleanup] Move methods to V8_DEPRECATED > > Updates ObjectTemplate::NewInstance and FunctionTemplate::GetFunction > from V8_DEPRECATED_SOON to V8_DEPRECATED, now that they're unused in > chrome. > > Bug: v8:7294, v8:7295, v8:8238 > Change-Id: Ic7cb2c410ff812f73cfd108551f2a1a20722df07 > Reviewed-on: https://chromium-review.googlesource.com/c/1344151 > Reviewed-by: Ulan Degenbaev <ulan@chromium.org> > Commit-Queue: Dan Elphick <delphick@chromium.org> > Cr-Commit-Position: refs/heads/master@{#57657} TBR=ulan@chromium.org,delphick@chromium.org Change-Id: I98a49bd0467f068b0122314ce2d0564dd174c2c5 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: v8:7294, v8:7295, v8:8238 Reviewed-on: https://chromium-review.googlesource.com/c/1344120Reviewed-by:
-
Clemens Hammacher authored
R=ahaas@chromium.org Bug: v8:8238 Change-Id: I7a7de894aa7bf074cbe732f40e16b10060fa37dd Reviewed-on: https://chromium-review.googlesource.com/c/1344149 Commit-Queue: Clemens Hammacher <clemensh@chromium.org> Reviewed-by:
Andreas Haas <ahaas@chromium.org> Cr-Commit-Position: refs/heads/master@{#57660}
-
Igor Sheludko authored
This CL also disables fast access to the embedder fields when pointer compression is enabled. Bug: v8:8477, v8:8238 Change-Id: I368bd87184590f8befdc882817497f75f0024569 Reviewed-on: https://chromium-review.googlesource.com/c/1344117Reviewed-by:
-
Clemens Hammacher authored
Instead of returning 0xDEADBEEF, return a struct with proper information. Otherwise a function returning 0xDEADBEEF would be misidentified as trapping in the interpreter. R=ahaas@chromium.org Bug: chromium:906997 Change-Id: I92fc3a9972d76d2f8a5b313bf6be6eb027cfc1e9 Reviewed-on: https://chromium-review.googlesource.com/c/1344111Reviewed-by:
-
Dan Elphick authored
Updates ObjectTemplate::NewInstance and FunctionTemplate::GetFunction from V8_DEPRECATED_SOON to V8_DEPRECATED, now that they're unused in chrome. Bug: v8:7294, v8:7295, v8:8238 Change-Id: Ic7cb2c410ff812f73cfd108551f2a1a20722df07 Reviewed-on: https://chromium-review.googlesource.com/c/1344151Reviewed-by:
-
Tobias Tebbi authored
By giving higher priority to processing non-effect-chain nodes, this changes the scheduling inside of basic blocks to place nodes before the latest possible effect chain node. Change-Id: I8b130904a1bb2360b995eb9de4f471a911a4e388 Reviewed-on: https://chromium-review.googlesource.com/c/1337743 Commit-Queue: Tobias Tebbi <tebbi@chromium.org> Reviewed-by:
-
Daniel Clifford authored
Includes creating Oddball-specific types. Change-Id: Ib5d80dfe95838b2deb0a2fcae67c349940970252 Reviewed-on: https://chromium-review.googlesource.com/c/1342930Reviewed-by:
-
Toon Verwaest authored
Change-Id: I13dbdbedf3d7bdc3488e9a6c3d4a1b0becfedd74 Reviewed-on: https://chromium-review.googlesource.com/c/1344116Reviewed-by:
-
Michael Starzinger authored
R=clemensh@chromium.org Change-Id: Ic56f6181a17520f54b9cab5dc3d106db56f1ede9 Reviewed-on: https://chromium-review.googlesource.com/c/1344115Reviewed-by:
-
