- 05 May, 2020 20 commits
-
-
Marja Hölttä authored
Bug: v8:9808 Change-Id: I24df18562c5b1277a8151d35eb07b836e70f1149 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2182457 Auto-Submit: Marja Hölttä <marja@chromium.org> Commit-Queue: Shu-yu Guo <syg@chromium.org> Reviewed-by:
Shu-yu Guo <syg@chromium.org> Cr-Commit-Position: refs/heads/master@{#67569}
-
Philip Pfaffe authored
Allow the DevTools frontend to evaluate variables in a wasm frame context by reusing the existing Debugger expression evaluation API. Where previously the API expected JavaScript expressions, which would in general just fail, now the expression is expected to be base64 encoded Wasm that creates a JSON string in linear memory. Bug: chromium:1020120 chromium:1068571 Change-Id: I4b31fdb9d3b21b4e08c4995ec2f07880923959e9 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2087396Reviewed-by:
Yang Guo <yangguo@chromium.org> Reviewed-by:
Simon Zünd <szuend@chromium.org> Reviewed-by:
Clemens Backes <clemensb@chromium.org> Commit-Queue: Philip Pfaffe <pfaffe@chromium.org> Cr-Commit-Position: refs/heads/master@{#67568}
-
Michael Lippautz authored
The flag is used by V8 to propagate information on whether this GC was somehow forced. This allows the embedder using EmbedderHeapTracer to easily figure out whether a GC was forced based on a local flag without depending on GC prologue/epilogue callbacks. Bug: chromium:1074061 Change-Id: Ic04c93ae58da854e54c6da1ca4aac03b5ab49897 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2182473Reviewed-by:
Ulan Degenbaev <ulan@chromium.org> Commit-Queue: Michael Lippautz <mlippautz@chromium.org> Cr-Commit-Position: refs/heads/master@{#67567}
-
Samuel Groß authored
Bug: v8:10391 Change-Id: I55d4d33820c83711d3ea3c6a2f3a20a36707fe36 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2151354Reviewed-by:
Jakob Gruber <jgruber@chromium.org> Reviewed-by:
Igor Sheludko <ishell@chromium.org> Reviewed-by:
Maya Lekova <mslekova@chromium.org> Commit-Queue: Igor Sheludko <ishell@chromium.org> Cr-Commit-Position: refs/heads/master@{#67566}
-
Hannes Payer authored
Change-Id: Ia316db16fb338e3f26b0666de88d5e53f375f8be Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2181263 Commit-Queue: Hannes Payer <hpayer@chromium.org> Reviewed-by:
-
Tobias Tebbi authored
Bug: chromium:1076708 Change-Id: I88a5eae0e562e32f1915deff3c4150ec4be14c6c Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2181266 Commit-Queue: Tobias Tebbi <tebbi@chromium.org> Commit-Queue: Georg Neis <neis@chromium.org> Auto-Submit: Tobias Tebbi <tebbi@chromium.org> Reviewed-by:
Georg Neis <neis@chromium.org> Cr-Commit-Position: refs/heads/master@{#67564}
-
Milad Farazmand authored
Without the added header some GCC compilers might produce this error: error: 'ceil' is not a member of 'std' Change-Id: I4fc784725c904adc58919a766671e37c9aca7ecb Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2181774 Commit-Queue: Milad Farazmand <miladfar@ca.ibm.com> Commit-Queue: Tobias Tebbi <tebbi@chromium.org> Reviewed-by:
Tobias Tebbi <tebbi@chromium.org> Cr-Commit-Position: refs/heads/master@{#67563}
-
Manos Koukoutos authored
Work towards adding heap-allocated object support for wasm, according to the gc proposal. Changes: - Implement subtyping for reference types (ref s) and (optref s), where 's' is a struct type. This CL does *not* implement subtyping between struct and function types. Also, it does not handle i31refs and eqrefs. - Implement struct.set. - Change struct.get to accept an optref as argument, as required by the standard. - Allow locals to store objects of ref and optref types. - Add a test for struct.set and optref locals. Modify the test for struct.get accordingly. Reference: https://github.com/WebAssembly/gc R=jkummerow@chromium.org R=clemensb@chromium.org Bug: v8:7748 Change-Id: I708626fa5f90a6e24e667d66eed1c7697f458a23 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2172089Reviewed-by:
Jakob Kummerow <jkummerow@chromium.org> Reviewed-by:
-
Georg Neis authored
This reverts commit a596efcc. Reason for revert: Was incorrect. Holes can appear in dead code. Original change's description: > [turbofan] Refine a DCHECK > > Hole checks are done using a lower level comparison. > > Change-Id: I61c5b787f12564ad3553d395a36938a00f5dd554 > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2172418 > Auto-Submit: Georg Neis <neis@chromium.org> > Commit-Queue: Nico Hartmann <nicohartmann@chromium.org> > Reviewed-by: Nico Hartmann <nicohartmann@chromium.org> > Cr-Commit-Position: refs/heads/master@{#67466} TBR=neis@chromium.org,nicohartmann@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Change-Id: I47aff68cf8e224882a3eeac0d9edfe5a6228f0f2 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2181324 Commit-Queue: Georg Neis <neis@chromium.org> Reviewed-by:
-
Thibaud Michaud authored
On intel platforms, floats do not always have a stable bit pattern. To preserve the bit pattern of float immediates, we should keep them in an int using get_bits() instead of casting them to a float with get_scalar(). R=ahaas@chromium.org CC=zhin@chromium.org Change-Id: I481219f755c40cbba760be5744734c1075e8ab66 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2172694 Commit-Queue: Thibaud Michaud <thibaudm@chromium.org> Reviewed-by:
Andreas Haas <ahaas@chromium.org> Cr-Commit-Position: refs/heads/master@{#67560}
-
Clemens Backes authored
This removes the interpreter entry stubs, which are used to redirect specific wasm functions to the interpreter. It is only needed when mixing JS code with interpreted Wasm code, otherwise the test functions just call the interpreter directly. Thus a lot of tests that contain such interaction between JS and Wasm need to be restricted to execute in Liftoff and TurboFan only. After this CL, the WASM_INTERPRETER_ENTRY frame type and the corresponding WasmInterpreterEntryFrame are dead, and will be removed in a follow-up CL. R=thibaudm@chromium.org Bug: v8:10389 Change-Id: I8e50d350dbc2afcc1cddaeb98baf23711117af2d Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2172962 Commit-Queue: Clemens Backes <clemensb@chromium.org> Reviewed-by:
Thibaud Michaud <thibaudm@chromium.org> Cr-Commit-Position: refs/heads/master@{#67559}
-
Samuel Groß authored
Bug: v8:10391 Change-Id: I29393ebcb58b1000040d7f7ba205895a8ba363f7 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2148782Reviewed-by:
-
Michael Lippautz authored
This adds PostConstructionCallbackTrait which can be used to get a callback that is executed right after an object instance is created. This can be useful for hooks that require to be able to call into virtual methods. Bug: chromium:1074061 Change-Id: Idd5ef677fed291bcba81b9a47f2932c9bb5832b4 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2179385 Commit-Queue: Michael Lippautz <mlippautz@chromium.org> Reviewed-by:
Anton Bikineev <bikineev@chromium.org> Reviewed-by:
Omer Katz <omerkatz@chromium.org> Cr-Commit-Position: refs/heads/master@{#67557}
-
Andreas Haas authored
The existing implementation needed uses of the outputs of an AtomicExchange to allocate registers for the result value. However, these uses are not guaranteed to exist. With this CL temp registers get allocated if the uses don't exist. R=gdeepti@chromium.org Bug: chromium:1077130 Change-Id: I058ee53b87c6e995c9f490f3aebbfdba69934f3c Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2179503Reviewed-by:
Deepti Gandluri <gdeepti@chromium.org> Commit-Queue: Andreas Haas <ahaas@chromium.org> Cr-Commit-Position: refs/heads/master@{#67556}
-
Shu-yu Guo authored
%GetUndetectable() is used for testing document.all's wacky "emulates undefined" behavior both in mjsunit tests and in test262. mjsunit doesn't rely its [[Call]] behavior, while test262 relies on its [[Call]] returning null [1]. So, make it return null. [1] https://github.com/tc39/test262/blob/master/INTERPRETING.md search for IsHTMLDDA Bug: v8:7184 Change-Id: I93d15715303deb0a932545d919bc281f5f4d5829 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2181475 Commit-Queue: Marja Hölttä <marja@chromium.org> Reviewed-by:
Marja Hölttä <marja@chromium.org> Cr-Commit-Position: refs/heads/master@{#67555}
-
Shu-yu Guo authored
R=marja@chromium.org Bug: v8:10372 Change-Id: Iaba823712140418b5a5ae73d77c50b4ea20df991 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2181164 Auto-Submit: Shu-yu Guo <syg@chromium.org> Commit-Queue: Marja Hölttä <marja@chromium.org> Reviewed-by:
-
Shu-yu Guo authored
There is already a %GetUndetectable runtime test function, so use that. Bug: v8:7184 Change-Id: I04af03d95c4245ab9c7061cb00d5890972b82f46 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2181195 Commit-Queue: Marja Hölttä <marja@chromium.org> Reviewed-by:
-
Shu-yu Guo authored
Bug: v8:9801 Change-Id: Id1c6aa94e291b9ea09515a60f248e0b3a0cc99c2 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2181163 Auto-Submit: Shu-yu Guo <syg@chromium.org> Commit-Queue: Marja Hölttä <marja@chromium.org> Reviewed-by:
-
Dan Elphick authored
Also makes memory-chunk.h accessible from outside heap which allows removal of some heap-inl.h includes. Bug: v8:10473, v8:10496 Change-Id: Iec4fc5ce8ad201f6ee5fd924cc3cd935324429fc Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2172088 Commit-Queue: Jakob Gruber <jgruber@chromium.org> Auto-Submit: Dan Elphick <delphick@chromium.org> Reviewed-by:
-
v8-ci-autoroll-builder authored
Rolling v8/build: https://chromium.googlesource.com/chromium/src/build/+log/81c45bb..8aef078 Rolling v8/third_party/aemu-linux-x64: KkC1dMnOt3dQMJqp-GzpAoyULRktX6-7fxUPrsEeEJMC..i0Kh0s13OUsO28Cn7E_Vm5Rv0jN7IXSm_qrvu2cBDmMC Rolling v8/third_party/catapult: https://chromium.googlesource.com/catapult/+log/72830df..cd56abf Rolling v8/third_party/depot_tools: https://chromium.googlesource.com/chromium/tools/depot_tools/+log/082a11a..0f47d33 TBR=machenbach@chromium.org,tmrts@chromium.org Change-Id: I8e06d3e7a3810f1ad892502adef37dfc6adb2bbf Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2182149Reviewed-by:
v8-ci-autoroll-builder <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Commit-Queue: v8-ci-autoroll-builder <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#67550}
-
- 04 May, 2020 20 commits
-
-
Frank Tang authored
Use the ICU 67.1 new API DateTimePatternGenerator::getDefaultHourCycle to replace a hack which get the pattern of "jjmm" to find out the default hour cycle of a locale Bump the required API version from 65 to 67 Bug: v8:10225 Change-Id: I3378edacb6dfb8400357ac0bf3d5d50b9fe008bd Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2173875Reviewed-by:
-
Frank Tang authored
The revised spec in https://tc39.es/ecma402/#sec-Intl.Locale.prototype.maximize now set the minimal or maximal to the %Locale% without the same opeartion as in Intl.Locale(tag, [option]) Bug: v8:10489 Change-Id: I08c45879b158a84e8cba19922423666e2b98412b Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2174976Reviewed-by:
-
Ng Zhi An authored
Bug: v8:10347 Change-Id: I5a64a9e90ec7e0f3f0baf032f2d6801a94c08a3d Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2168026Reviewed-by:
Benedikt Meurer <bmeurer@chromium.org> Reviewed-by:
-
Deepti Gandluri authored
This reverts commit 3c400825. Reason for revert: Re-enable interpreter tests Original change's description: > [wasm-simd] Remove interpreter tier of SIMD tests > > As per the all-hands a couple of weeks ago, the interpreter will > be removed soon. Remove running tests on this tier, so we no longer > put effort into maintaining tests for this tier. > > Change-Id: I9fce0f3a7cd869d6ccecf1c1f820b794e89858e1 > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2175021 > Reviewed-by: Zhi An Ng <zhin@chromium.org> > Commit-Queue: Deepti Gandluri <gdeepti@chromium.org> > Cr-Commit-Position: refs/heads/master@{#67520} TBR=gdeepti@chromium.org,zhin@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Change-Id: Iac0f21311769157c5ae303e8078c25d96fbc7c93 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2180343Reviewed-by:
Zhi An Ng <zhin@chromium.org> Commit-Queue: Deepti Gandluri <gdeepti@chromium.org> Cr-Commit-Position: refs/heads/master@{#67546}
-
Dan Elphick authored
This removes the static ReadOnlyHeap::Instance method replacing it with PopulateReadOnlySpaceStatistics on the way to removing the global ReadOnlyHeap object. Bug: v8:10454 Change-Id: Ic78c898ff99c6a7dac023d2b5230fbbbf6f36f46 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2179805 Commit-Queue: Dan Elphick <delphick@chromium.org> Commit-Queue: Ulan Degenbaev <ulan@chromium.org> Auto-Submit: Dan Elphick <delphick@chromium.org> Reviewed-by:
-
Bill Budge authored
- Implements WasmInt32ToHeapNumber, WasmTaggedNonSmiToInt32, and WasmTaggedToFloat64 as Torque builtins. Bug: v8:10070 Change-Id: I8b16d000b5283f27f7762341e9dbbaf5ab3ebb62 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2173395Reviewed-by:
-
Thibaud Michaud authored
This allows us to preserve the script URL when importing a module in a worker. R=ahaas@chromium.org,clemensb@chromium.org CC=kimanh@chromium.org Bug: chromium:1064548 Change-Id: Id5e48c840e2dba8eadb5c854fcb389787ce11215 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2167866 Commit-Queue: Thibaud Michaud <thibaudm@chromium.org> Reviewed-by:
-
Tobias Tebbi authored
Summary of changes: - GC visitors no longer rely on superclass visitors, but instead visit everything themselves. This enables generating better code. - Try to match simple body descriptors to reduce the amount of generated code. - Turn SizeFor(instance) into an AllocatedSize() method. - Remove the special handling of resizable object sizes from Torque and instead overwrite AllocatedSize in classes that need special handling in C++. - Split the visitor id lists depending on whether the class has pointer fields. - Turn Torque-generated body descriptors into an .inc file to simplify includes. - Fix generated size functions to properly align the size. - Generate GC visitors (and C++ class definitions) for all string classes and FixedArray, WeakFixedArray, and WeakArrayList. - Store generated instance types in Torque class types. This is only used to determine if a type has a single instance type in this CL. Bug: v8:7793 Change-Id: I4d362e96b047c305bd6d065247734957b8958c42 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2110014 Commit-Queue: Tobias Tebbi <tebbi@chromium.org> Reviewed-by:
Nico Hartmann <nicohartmann@chromium.org> Reviewed-by:
-
Igor Sheludko authored
... to be in sync with KeyedLoadIC_SloppyArguments in handling OOB accesses which may involve prototype chain walk. Bug: chromium:1063796 Change-Id: I8421c19085dfd2f3b6360c64fd04f53b1351576c Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2174504Reviewed-by:
Toon Verwaest <verwaest@chromium.org> Commit-Queue: Igor Sheludko <ishell@chromium.org> Cr-Commit-Position: refs/heads/master@{#67541}
-
Michael Lippautz authored
The detection was overbroad and the underlying issue in QueryPerformanceCounter is supposedly fixed from Win XP SP2 on. Chromium removed the detection in https://crrev.com/c/1138241 in July 2018. Bug: v8:10362 Change-Id: I87baa47c2aad2428a92923a60916c0a9d07afdad Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2128052Reviewed-by:
-
Santiago Aboy Solanes authored
Since not all uses are going to be needing a revisit, we can introduce additional bookkeeping to search in the subset does need it. Sadly, it can only be used during the Visit part of RETYPE, since during the revisit all uses might need to be revisited. Bug: v8:10424 Change-Id: I4650ea42a93316d54de7d3aa32ce8a5eef2e10e4 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2139573 Commit-Queue: Santiago Aboy Solanes <solanes@chromium.org> Reviewed-by:
-
Daniel Bevenius authored
Change-Id: I4336c761cbf04800e07b2533be38265725a0d7a2 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2179302Reviewed-by:
-
Toon Verwaest authored
Previously such maps were marked as prototype, but that has bad performance / memory characteristics if objects are used as dictionaries. Bug: b:148346655, v8:10339 Change-Id: I287c5664c8b7799a084669aaaffe3affcf73e95f Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2179322Reviewed-by:
-
Jakob Gruber authored
This variant passes the --stress-snapshot d8 flag. There's a large initial list of skips, these should be removed as issues are fixed over time. The variant is currently not enabled on any bots. Bug: v8:10416 Change-Id: I80aea80600c51b2f5d28b8ec8a09ff0ba2ebaa7a Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2179002 Commit-Queue: Jakob Gruber <jgruber@chromium.org> Reviewed-by:
Michael Achenbach <machenbach@chromium.org> Cr-Commit-Position: refs/heads/master@{#67536}
-
Jakob Gruber authored
... and weak context code lists. These are non-empty when an isolate is running and the serializer cannot handle them. Bug: v8:10416 Change-Id: I11a3d25dfd1980bcddae8b65c429df3c2cf16b19 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2172423 Commit-Queue: Jakob Gruber <jgruber@chromium.org> Reviewed-by:
Dan Elphick <delphick@chromium.org> Cr-Commit-Position: refs/heads/master@{#67535}
-
Marja Hölttä authored
The objects are already in fast mode so they were NOOP. The commit that obsoleted them was https://chromium-review.googlesource.com/c/v8/v8/+/571750/ Bug: v8:10479,v8:5902 Change-Id: Ic3611f899d3138cda5bcff6debe529118ced5c93 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2172746 Commit-Queue: Marja Hölttä <marja@chromium.org> Reviewed-by:
-
Igor Sheludko authored
Bug: v8:10391 Change-Id: I316a3c5cd986a74d7f46da6d0b85cb3d549be497 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2153209 Commit-Queue: Igor Sheludko <ishell@chromium.org> Reviewed-by:
-
Michael Lippautz authored
The callback is useful to embedders to trigger various actions such as recomputing live set. Bug: chromium:1056170 Change-Id: I7d80b9b768a728e23303f945e416df97fd9b7805 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2173358 Commit-Queue: Michael Lippautz <mlippautz@chromium.org> Reviewed-by:
-
Dan Elphick authored
Moves ReadOnlyPage, ReadOnlyArtifacts, ReadOnlySpace and SharedReadOnlySpace out of spaces.h and into read-only-spaces.h, as well as creating a corresponding .cc file. Bug: v8:10473 Change-Id: I9d8b49d61ed643fd6e16919d571a909ab6fce407 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2171197Reviewed-by:
-
Manos Koukoutos authored
Motivation: Improve code efficiency by replacing runtime calls with manually written turbofan code where possible. Changes: - Remove the runtime functions `Runtime_WasmExceptionGetTag` and `Runtime_WasmExceptionGetValues` and replace them with turbofan code. - Introduce the builtin function `GetOwnProperty`. - Change `wasm-compiler.h` and `wasm-compiler.cc` to accomodate the new changes. - Introduce three new macros in `wasm-compiler.cc`. - Use those macros in two additional places to remove code duplication. Change-Id: I4a32f9e5f7ee55dc50cd03378a68897888ece5c8 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2162905Reviewed-by:
-
