- 21 Nov, 2018 13 commits
-
-
Camillo Bruni authored
No-Try: true Change-Id: I078279bef7141c0255025efebfd3a39a57fc3bf7 Reviewed-on: https://chromium-review.googlesource.com/c/1345149Reviewed-by:
Michael Lippautz <mlippautz@chromium.org> Commit-Queue: Camillo Bruni <cbruni@chromium.org> Cr-Commit-Position: refs/heads/master@{#57674}
-
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:
Michael Achenbach <machenbach@chromium.org> Commit-Queue: Clemens Hammacher <clemensh@chromium.org> Cr-Commit-Position: refs/heads/master@{#57671}
-
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:
Jaroslav Sevcik <jarin@chromium.org> Reviewed-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:
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:
Tobias Tebbi <tebbi@chromium.org> Cr-Commit-Position: refs/heads/master@{#57667}
-
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:
Sigurd Schneider <sigurds@chromium.org> Reviewed-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:
Benedikt Meurer <bmeurer@chromium.org> Reviewed-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 27 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:
Igor Sheludko <ishell@chromium.org> Commit-Queue: Igor Sheludko <ishell@chromium.org> Cr-Commit-Position: refs/heads/master@{#57661}
-
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:
Ulan Degenbaev <ulan@chromium.org> Commit-Queue: Igor Sheludko <ishell@chromium.org> Cr-Commit-Position: refs/heads/master@{#57659}
-
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:
Clemens Hammacher <clemensh@chromium.org> Commit-Queue: Michael Starzinger <mstarzinger@chromium.org> Cr-Commit-Position: refs/heads/master@{#57653}
-
Georg Neis authored
R=bmeurer@chromium.org Change-Id: I215faf30fcd6c0a7c67a4969ec9255dd11b9be08 Reviewed-on: https://chromium-review.googlesource.com/c/1344112Reviewed-by:
-
Michael Starzinger authored
This test cases ensures that it is possible to maintain a coherent instruction cache by using {Assembler::FlushICache} in any order with respect to changing page permissions via {SetPermissions}. R=ulan@chromium.org TEST=cctest/test-icache BUG=v8:6792 Change-Id: I1778bbbe94b40856d9732a9990a931b5a956a439 Reviewed-on: https://chromium-review.googlesource.com/c/1344113Reviewed-by: -
Sergiy Byelozyorov authored
This reverts commit f401cd4b. Reason for revert: did not help Original change's description: > [tools] Re-land: Add retries when trying to discover the device > > This is an experimental change that may help mitigate the issue. > > TBR=machenbach@chromium.org > > No-Try: true > No-Tree-Checks: true > Bug: chromium:893593 > Change-Id: Idf15a63006c2c7ba2c31482e5103b2a0b1d64510 > Reviewed-on: https://chromium-review.googlesource.com/c/1339401 > Commit-Queue: Sergiy Byelozyorov <sergiyb@chromium.org> > Reviewed-by: Sergiy Byelozyorov <sergiyb@chromium.org> > Cr-Commit-Position: refs/heads/master@{#57558} TBR=machenbach@chromium.org,sergiyb@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: chromium:893593 Change-Id: I30c529a627d1e6fa52099939c5c209110e9d0eb9 Reviewed-on: https://chromium-review.googlesource.com/c/1342931Reviewed-by:
Sergiy Byelozyorov <sergiyb@chromium.org> Reviewed-by:
-
Georg Neis authored
This is a reland of 9c91b687 after fixing undefined behavior in numeric conversion that caused trouble on arm32. Original change's description: > [turbofan] Use feedback when reducing global loads/stores. > > We already record the script context location or the property cell > as feedback of the global load/store IC, so Turbofan doesn't need > to do the lookups again. > > Change-Id: I6cbd2937de344729cd8e146b4ff85ddf3de6a56e > Reviewed-on: https://chromium-review.googlesource.com/c/1335691 > Commit-Queue: Georg Neis <neis@chromium.org> > Reviewed-by: Benedikt Meurer <bmeurer@chromium.org> > Cr-Commit-Position: refs/heads/master@{#57555} Change-Id: Ic2d09025de02f92199755ac860bb9e91fa08f4ec Reviewed-on: https://chromium-review.googlesource.com/c/1340043Reviewed-by:
-
Mythri authored
This cl updates: 1. Adds a new feedback cell map to specify that no feedback is collected 2. Checks if feedback vectors are valid before using then when creating closures 3. Runtime profiler to only tier up functions with feedback 4. Interpreter entry trampoline to check for feedback vector before using it. Bug: v8:8394 Change-Id: I0248c8cd35d841c2744b22f4c672fa2e82033f6e Reviewed-on: https://chromium-review.googlesource.com/c/1339866 Commit-Queue: Mythri Alle <mythria@chromium.org> Reviewed-by:
Ross McIlroy <rmcilroy@chromium.org> Reviewed-by:
-
Toon Verwaest authored
Change-Id: I08af90a6bd863402309c1c65c829db6a955c67e3 Reviewed-on: https://chromium-review.googlesource.com/c/1342927Reviewed-by:
-
Clemens Hammacher authored
This adds a {PrintRegister} method which prints the given register in a readable way (e.g. "eax", ... on ia32). This is currently only used in Liftoff. The {RegisterConfiguration} class has the same functionality, and I plan to make {RegisterConfiguration} also use the new {RegisterName} functions in a follow-up CL. R=mstarzinger@chromium.org Bug: v8:8238, v8:8423, v8:6600 Change-Id: If03901f1d8c5b043e0097e63920ab711bd7e2d17 Reviewed-on: https://chromium-review.googlesource.com/c/1340041Reviewed-by: -
Tobias Tebbi authored
This enables more seamless interop between Torque and CSA: Since CodeStubAssembler can now inherit from the Torque base namespace, macros defined in the base namespace can be used in CodeStubAssembler macros, even without qualification. At the same time, macros in the base namespace can refer to CodeStubAssembler macros. The only new limitation is that types defined in code-stub-assembler.h cannot be referenced in the signature of macros defined in the base namespace, since this would produce a cyclic header dependency. A work-around for this woud be to put such types (like int31 in this CL) into a separate header included by both. I (mis-)used code-assembler.h for that. Another side-effec is that types and enums defined in CodeStubAssembler have to be accessed in a qualified way from Torque. Other assemblers can now inherit from their Torque equivalent, so porting macros into the corresponding Torque namespace doesn't require any change to the existing use-sites. To avoid C++ ambiguities, the Torque-generated assemblers must not define anything also defined in Code(Stub)Assembler. This includes the type aliases for TNode, PLabel, ... My workaround is to qualify everything in the generated C++. As a drive-by fix, I had to change the formatter to avoid a situation where it doesn't compute a fixed point: putting a keyword at the beginning of a line removes the '\s' in front of it, so I replaced that with '\b'. Bug: v8:7793 Change-Id: If3b9e9ad967a181b380a10d5673615606abd1041 Reviewed-on: https://chromium-review.googlesource.com/c/1341955Reviewed-by:
-
Michael Lippautz authored
This is a reland of b1468242. Bug: chromium:843903, chromium:903586 Change-Id: Ida59ba4efd3abae6956b99aa104bbc66a3f01fdc Reviewed-on: https://chromium-review.googlesource.com/c/1342924Reviewed-by:
-
Michael Achenbach authored
NOTRY=true Bug: v8:8457 Change-Id: I9893f675232f47ae1529fd773adb524f60418c92 Reviewed-on: https://chromium-review.googlesource.com/c/1342928Reviewed-by:
-
Yang Guo authored
R=jgruber@chromium.org Change-Id: I07d9c2f952f0ca3cf50f5faa4ad8db78be2a5b04 Reviewed-on: https://chromium-review.googlesource.com/c/1342926Reviewed-by:
Jakob Gruber <jgruber@chromium.org> Commit-Queue: Yang Guo <yangguo@chromium.org> Cr-Commit-Position: refs/heads/master@{#57642}
-
Pierre Langlois authored
Emitting unwinding info for builtins for perf to consume doesn't make sense with embedded builtins so let's just remove the option. The perf support is meant for code on the heap and the builtins are not there anymore. If we want perf to be able to unwind through builtins we should emit the unwinding DWARF information directly into the binary, using the dedicated .eh_frame ELF section. This would also mean GDB would be able to unwind through builtins as well which would be great. Change-Id: I751cc5eb1e6f7c0eeae6b37a42986ae8ea47d6a0 Reviewed-on: https://chromium-review.googlesource.com/c/1340294Reviewed-by:
Michael Stanton <mvstanton@chromium.org> Reviewed-by:
-
Clemens Hammacher authored
Just compute it from the number of outstanding units. R=ahaas@chromium.org Bug: v8:7921 Change-Id: I30db10accc032bc50e1bbeab599325e1e971972b Reviewed-on: https://chromium-review.googlesource.com/c/1341953 Commit-Queue: Clemens Hammacher <clemensh@chromium.org> Reviewed-by:
-
Igor Sheludko authored
for storing embedder data in native context. We can't use FixedArray because with enabled pointer compression it would not be possible to fit raw aligned pointer into 32-bits of a tagged value so we will need to store both tagged and raw data in this array and therefore custom visitor is required. Bug: v8:7703 Change-Id: Iae23d9aa76c79a572d5f0f1f3c0f924e8e407dd0 Reviewed-on: https://chromium-review.googlesource.com/c/1340295 Commit-Queue: Igor Sheludko <ishell@chromium.org> Reviewed-by:
-
Ulan Degenbaev authored
The concurrent marker needs to access the kind_specific_flags to decide whether an embedded object reference is weak or not. This patch turns the Code::code_data_container() into an acquire/release atomic accessor and makes CodeDataContainer::kind_specific_flags a relaxed atomic accessor. Bug: v8:8459 Change-Id: I5251fed4e7b3315f8e229dfcfe2c23f611f4b333 Reviewed-on: https://chromium-review.googlesource.com/c/1337746Reviewed-by:
-
Georg Neis authored
This has been on by default and the flag itself is no longer useful. R=jarin@chromium.org, mslekova@chromium.org Bug: v8:7790 Change-Id: Icdf111b974a01953ea775ccb96d50217f3c8321b Reviewed-on: https://chromium-review.googlesource.com/c/1342918 Commit-Queue: Georg Neis <neis@chromium.org> Reviewed-by:
Maya Lekova <mslekova@chromium.org> Reviewed-by:
-
Andreas Haas authored
In the chrome-side implementation I currently use the default trap handlers of V8, see https://crrev.com/c/1290955 Bug: chromium:906565 Cq-Include-Trybots: luci.chromium.try:linux_chromium_rel_ng Change-Id: I74c5a18c479ad1c69303d104ad4f040de436c4e7 Reviewed-on: https://chromium-review.googlesource.com/c/1282960 Commit-Queue: Andreas Haas <ahaas@chromium.org> Reviewed-by:
Ben Titzer <titzer@chromium.org> Reviewed-by:
-
Toon Verwaest authored
If we're hashing a single sequential string we don't need the state that the string hasher itself tracks. This also drops first_char since we can simply check that array_index is still 0. Change-Id: Icb69709267426358f7c301eeb45936843ba261b0 Reviewed-on: https://chromium-review.googlesource.com/c/1340258 Commit-Queue: Toon Verwaest <verwaest@chromium.org> Reviewed-by:
-
