- 23 Nov, 2020 16 commits
-
-
bcoe authored
Introduces getEnclosingColumn and getEnclosingLine on CallSite so that the position can be used to lookup the original symbol for function when source maps are used. BUG=v8:11157 Change-Id: I06c4c374d172d206579abb170c7b7a2bd3bb159f Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2547218Reviewed-by:
Jakob Kummerow <jkummerow@chromium.org> Commit-Queue: Benjamin Coe <bencoe@google.com> Cr-Commit-Position: refs/heads/master@{#71343}
-
Camillo Bruni authored
- Add support for module streaming compilation - Enable module streaming testing d8 - Update API tests to include basic module streaming Bug: chromium:1061857 Change-Id: I3ac95f7d672c382406182fb6900b1095f15c63b3 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2536457Reviewed-by:
Toon Verwaest <verwaest@chromium.org> Reviewed-by:
Leszek Swirski <leszeks@chromium.org> Commit-Queue: Camillo Bruni <cbruni@chromium.org> Cr-Commit-Position: refs/heads/master@{#71342}
-
Victor Gomes authored
ConstructWithSpread_WithFeedback uses the same argument order as JS linkage, therefore arguments should be inserted in reversed order. See https://source.chromium.org/chromium/chromium/src/+/master:v8/src/codegen/interface-descriptors.h;drc=c7cb9beca18d98ba83c3b75860b912219d425d0e;l=507 Change-Id: I4d3ded048a08ba9a2a4d30da4c41044d9669becc Bug: chromium:1145990 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2549952 Commit-Queue: Victor Gomes <victorgomes@chromium.org> Reviewed-by:
Jakob Gruber <jgruber@chromium.org> Cr-Commit-Position: refs/heads/master@{#71341}
-
Santiago Aboy Solanes authored
If we have a regular isolate (or none at all), we can skip acquiring the lock check and DCHECK that we are calling from the main thread. If we have a LocalIsolate, we acquire the string lock if needed. Bug: v8:7790 Change-Id: Ie3562e8172a3e3eca8d194e8652cb881f765cdb8 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2551102 Commit-Queue: Santiago Aboy Solanes <solanes@chromium.org> Reviewed-by:
Ross McIlroy <rmcilroy@chromium.org> Reviewed-by:
Ulan Degenbaev <ulan@chromium.org> Cr-Commit-Position: refs/heads/master@{#71340}
-
Manos Koukoutos authored
We use the same temporary mechanism as with eqref, in anticipation of standardization of the wasm-gc JS API. Bug: v8:7748 Change-Id: I224a043e5450ce489fc7f3b2f07f277a0444b8e0 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2546695 Commit-Queue: Manos Koukoutos <manoskouk@chromium.org> Reviewed-by:
-
Dominik Inführ authored
Change-Id: I51f2152d8a26fb0b266a41f7d284ced7908eb475 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2554603 Commit-Queue: Dominik Inführ <dinfuehr@chromium.org> Commit-Queue: Ulan Degenbaev <ulan@chromium.org> Auto-Submit: Dominik Inführ <dinfuehr@chromium.org> Reviewed-by:
-
Clemens Backes authored
This changes a '<' to a '<=' and adds a comment to explain why it is safe to use a jump table where the maximum distance is exactly {kMaxCodeSpaceSize}. R=jkummerow@chromium.org Bug: chromium:1151364 Change-Id: Id4971a2e9095fa99df48367ab09af4adbfadffaf Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2552906Reviewed-by: -
Liu Yu authored
Port: 40c0f84a Bug: v8:9771 Change-Id: Icbe4e3450bb6ef7242804ca9d7f46cb6f1aed40c Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2539538 Auto-Submit: Liu yu <liuyu@loongson.cn> Reviewed-by:
Zhao Jiazhong <zhaojiazhong-hf@loongson.cn> Commit-Queue: Zhao Jiazhong <zhaojiazhong-hf@loongson.cn> Cr-Commit-Position: refs/heads/master@{#71336}
-
Jakob Kummerow authored
For a very particular special case (long "chains" of bound functions with an undefined @@hasInstance handler), evaluating the `instanceof` operator could lead to a very deep recursion. This patch adds a stack check to make sure we throw rather than crash on stack overflow. Bug: v8:11115 Change-Id: I6bf941b9e75e9fe3a52112ade27388ac4fbbda2f Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2545624Reviewed-by:
Shu-yu Guo <syg@chromium.org> Commit-Queue: Jakob Kummerow <jkummerow@chromium.org> Cr-Commit-Position: refs/heads/master@{#71335}
-
Georg Neis authored
Bug: v8:7790 Change-Id: Idee149b3d59064941473d5e17e2c56a253a5f49d Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2546691 Auto-Submit: Georg Neis <neis@chromium.org> Reviewed-by:
Santiago Aboy Solanes <solanes@chromium.org> Commit-Queue: Georg Neis <neis@chromium.org> Cr-Commit-Position: refs/heads/master@{#71334}
-
Michael Achenbach authored
Don't pass the correctness-fuzzing suppressions to normal fuzzing as they turn stack overflows and invalid string length checks into crashes. This became first now a problem after the flag was passed in an mjsunit test case. No-Try: true Bug: chromium:1151600,chromium:1151599 Change-Id: I5d29900a4b155762cae447fc102055eab1916309 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2551112 Auto-Submit: Michael Achenbach <machenbach@chromium.org> Commit-Queue: Clemens Backes <clemensb@chromium.org> Reviewed-by:
Clemens Backes <clemensb@chromium.org> Cr-Commit-Position: refs/heads/master@{#71333}
-
Zhao Jiazhong authored
I'm working for Loongson Technology, and I have contributed 120+ patches to maintain v8 on mips platform. I request to be an owner of MIPS files, so that we can maintain mips ports more conveniently. Change-Id: Ib01dadfb879fefe7c095398930573e8df0f7c8dd Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2525542 Commit-Queue: Zhao Jiazhong <zhaojiazhong-hf@loongson.cn> Reviewed-by:
Hannes Payer <hpayer@chromium.org> Cr-Commit-Position: refs/heads/master@{#71332}
-
Liu Yu authored
Port: commit 1da429fb Bug: v8:10949 Change-Id: I77d28b26a78fe098b529d6ac333c0dac49850b4f Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2553160Reviewed-by:
-
Marja Hölttä authored
When comparing two-byte strings, the correct number of characters to compare is length(), not byte_length(). The bug was introduced in https://chromium-review.googlesource.com/c/v8/v8/+/2533038 There's no regression test, since going beyond the AstRawString boundary generally doesn't crash. Bug: chromium:1151602 Change-Id: I32c297c2751835dd7574ff928d2d5b8346b4381a Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2551110Reviewed-by:
-
Camillo Bruni authored
Change-Id: Ib34bb9aeeab0e092cdfa49da11382cdb5a8a20e3 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2545709Reviewed-by:
-
v8-ci-autoroll-builder authored
Rolling v8/build: https://chromium.googlesource.com/chromium/src/build/+log/afc55ca..49ce9a3 TBR=machenbach@chromium.org,tmrts@chromium.org,v8-waterfall-sheriff@grotations.appspotmail.com Change-Id: I70102bf08d91bdc1503e1dd8160dbee9b252bf7a Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2553924Reviewed-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@{#71328}
-
- 22 Nov, 2020 1 commit
-
-
v8-ci-autoroll-builder authored
Rolling v8/build: https://chromium.googlesource.com/chromium/src/build/+log/030a312..afc55ca Rolling v8/third_party/aemu-linux-x64: gt2DKWmtJU6vqOju1UcBB-_Nthud81s3cnZkERzzSEUC..nv6wFuL5e4oM14o83fKYTaYGvYpeIY0g-cCj2yzejZwC Rolling v8/third_party/catapult: https://chromium.googlesource.com/catapult/+log/11b4013..91c1a7c TBR=machenbach@chromium.org,tmrts@chromium.org,v8-waterfall-sheriff@grotations.appspotmail.com Change-Id: Ife95420b8e3b3c42a473f37bf3518c0323736200 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2553666Reviewed-by:
-
- 21 Nov, 2020 2 commits
-
-
v8-ci-autoroll-builder authored
Rolling v8/build: https://chromium.googlesource.com/chromium/src/build/+log/ee1c001..030a312 Rolling v8/third_party/aemu-linux-x64: 4f-YWuHlGrqS9jy308GUs0eo8DxU3h6PwgpHfNYq290C..gt2DKWmtJU6vqOju1UcBB-_Nthud81s3cnZkERzzSEUC Rolling v8/third_party/catapult: https://chromium.googlesource.com/catapult/+log/2ed6fc0..11b4013 Rolling v8/third_party/depot_tools: https://chromium.googlesource.com/chromium/tools/depot_tools/+log/2f8e0fa..9c0dc30 Rolling v8/third_party/zlib: https://chromium.googlesource.com/chromium/src/third_party/zlib/+log/e84c9a3..9893e50 Rolling v8/tools/luci-go: git_revision:1a022d3a4c50be4207ee93451255d71896416596..git_revision:6cbe3f56e9f00b8f65eae21f01838a8b58191a47 Rolling v8/tools/luci-go: git_revision:1a022d3a4c50be4207ee93451255d71896416596..git_revision:6cbe3f56e9f00b8f65eae21f01838a8b58191a47 Rolling v8/tools/luci-go: git_revision:1a022d3a4c50be4207ee93451255d71896416596..git_revision:6cbe3f56e9f00b8f65eae21f01838a8b58191a47 TBR=machenbach@chromium.org,tmrts@chromium.org,v8-waterfall-sheriff@grotations.appspotmail.com Change-Id: Ia6309934987cdc0f0da95a83875041761673ec3d Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2553156Reviewed-by:
-
Milad Fa authored
Without the cast, gcc might throw the following error during compilation: error: enumeral mismatch in conditional expression: 'cppgc::internal::StatsCollector::ScopeId' vs 'cppgc::internal::StatsCollector::ConcurrentScopeId' Change-Id: I95e230310a0cbdc775d63657b8c407a8392a57e3 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2551104Reviewed-by:
Omer Katz <omerkatz@chromium.org> Commit-Queue: Milad Fa <mfarazma@redhat.com> Cr-Commit-Position: refs/heads/master@{#71325}
-
- 20 Nov, 2020 21 commits
-
-
Junliang Yan authored
Drive-by: Add alias for lzer on Assembler Change-Id: Id0d705ef864899241f77d92c2cf8a144f753ef15 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2552928Reviewed-by:
Milad Fa <mfarazma@redhat.com> Commit-Queue: Junliang Yan <junyan@redhat.com> Cr-Commit-Position: refs/heads/master@{#71324}
-
Junliang Yan authored
Change-Id: I771d3db6510146b043c9dd0a3bae8a9a67b21176 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2552927Reviewed-by:
-
Thibaud Michaud authored
Following up on: [wasm] Deserialization: copy and relocate in the background https://chromium-review.googlesource.com/c/v8/v8/+/2543932 Also move publishing in a background task. Now all three steps of deserialization run in separate task as a pipeline. R=ahaas@chromium.org CC=clemensb@chromium.org Bug: v8:11164 Change-Id: Ic4ad09ea5e96bda84cc66d0bdc6473aadea5596b Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2551101 Commit-Queue: Thibaud Michaud <thibaudm@chromium.org> Reviewed-by:
Andreas Haas <ahaas@chromium.org> Cr-Commit-Position: refs/heads/master@{#71322}
-
Junliang Yan authored
Change-Id: I63c10010a9605f1ab40b9ce00039aa6a6a46bbbf Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2552545Reviewed-by:
-
Andreas Haas authored
With pointer compression, the size of a reference depends on whether it is stored on the stack or on the heap. The size provided by ValueType::element_size_bytes() is the size of a reference on the heap. LiftoffAssembler::SlotSizeForType(...) however should return the size on the stack. This CL fixes this inconsistency. This issue would have been found by an existing test, but this test is disabled at the moment because of missing safepoint maps for stack checks. R=thibaudm@chromium.org Bug: v8:7581 Change-Id: Ia45944b265fa4ce0d560ff00a24b023d6c1ae10a Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2552515Reviewed-by:
Thibaud Michaud <thibaudm@chromium.org> Commit-Queue: Andreas Haas <ahaas@chromium.org> Cr-Commit-Position: refs/heads/master@{#71320}
-
Leszek Swirski authored
Because of LocalHeap safepoints, our existing assert scopes don't necessarily maintain the same guarantees as desired. In particular, DisallowHeapAllocation no longer guarantees that objects don't move. This patch transitions DisallowHeapAllocation to DisallowGarbageCollection, to ensure that code using this scope is also protected against safepoints. Change-Id: I0411425884f6849982611205fb17bb072881c722 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2540547 Commit-Queue: Leszek Swirski <leszeks@chromium.org> Reviewed-by:
Peter Marshall <petermarshall@chromium.org> Reviewed-by:
Tobias Tebbi <tebbi@chromium.org> Reviewed-by:
Igor Sheludko <ishell@chromium.org> Cr-Commit-Position: refs/heads/master@{#71319}
-
Alex Turner authored
This retrieves script name directly from StackFrameBase, bypassing building of StackFrameInfo if one hasn't already been initialized, thus avoiding computation of expensive properties that are not required. This matches current behavior of GetScriptNameOrSourceURL() and is a workaround until a dedicated API is available. This is necessary to switch AdTagging over from using GetScriptNameOrSourceURL() to GetScriptName(), to ensure that scripts with source urls are tagged appropriately. (See crrev.com/c/2551259.) Bug: chromium:1127391 Change-Id: I6eb145b88c26deb1a088f038b0f8b377bc8fe3ab Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2550504Reviewed-by:
Simon Zünd <szuend@chromium.org> Commit-Queue: Alex Turner <alexmt@chromium.org> Cr-Commit-Position: refs/heads/master@{#71318}
-
Milad Fa authored
Port 1da429fb Original Commit Message: This is a reland of 800307f6, with a minimal fix for arm64 (uint64_t -> uintptr_t). Original change's description: > [wasm][memory64] Prepare Liftoff for ptrsize offsets > > This CL prepares the LiftoffAssembler interface for uintptr_t offsets. > Many places can still only handle 32-bit values, but after this CL we can > start storing the offsets as uintptr_t in the memory access immediates. > Some TODOs are placed to extend code generation for 64-bit additions, if > memory64 is enabled. > All of this will be addressed in follow-up CLs. > > R=manoskouk@chromium.org > > Bug: v8:10949 > Change-Id: Id3b9b8aa555ab41f082ba012f4f8d80586c35b89 > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2529452 > Commit-Queue: Clemens Backes <clemensb@chromium.org> > Reviewed-by: Manos Koukoutos <manoskouk@chromium.org> > Cr-Commit-Position: refs/heads/master@{#71236} R=clemensb@chromium.org, joransiu@ca.ibm.com, junyan@redhat.com, midawson@redhat.com BUG= LOG=N Change-Id: I87a421ab1fe6e4d0f2098c24ff34a3888631722e Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2552166Reviewed-by:
Junliang Yan <junyan@redhat.com> Commit-Queue: Milad Fa <mfarazma@redhat.com> Cr-Commit-Position: refs/heads/master@{#71317}
-
Michael Lippautz authored
Bug: chromium:1056170 Change-Id: I53e0e45045282c1fd217af34ff31b7e6411624b1 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2552508 Commit-Queue: Michael Lippautz <mlippautz@chromium.org> Commit-Queue: Ulan Degenbaev <ulan@chromium.org> Auto-Submit: Michael Lippautz <mlippautz@chromium.org> Reviewed-by:
-
Vasili Skurydzin authored
Change-Id: I6b5676477cc06ef6a842d80ef2ba6e00265f73dc Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2547206Reviewed-by:
-
Andreas Haas authored
CalculateGlobalOffsets has to be called once after all globals got defined but before global offsets get accessed, e.g. during code generation. It is not clear though when exactly CalculateGlobalOffsets should be called. The globals section may not exist, so at the end of the globals section is not enough (globals can also be defined in the import section). At the beginning of the code section is also not good enough, because the code section may not exist. At the end of the module may be too late. With this CL, CalculateGlobalOffsets is called after the global section, before the code section, and at the end of the module. Additionally the CL checks if CalculateGlobalOffsets has already been called, so that it is not executed a second time. R=manoskouk@chromium.org Bug: v8:11185 Change-Id: I922b9f60a4a17a09d2527fd9ab35cda71226030c Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2551100 Commit-Queue: Andreas Haas <ahaas@chromium.org> Reviewed-by:
Manos Koukoutos <manoskouk@chromium.org> Cr-Commit-Position: refs/heads/master@{#71314}
-
Mythri A authored
Bug: v8:6949, v8:11074 Change-Id: I05326fd514ea33ccaa783f5964a338d84b370e84 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2550781Reviewed-by:
-
Manos Koukoutos authored
Changes: - Move enhancement of locals_count by 1 inside AnalyzeLoopAssignment. - Update documentation of AnalyzeLoopAssignment. - Factor out invocation to OpcodeLength(); - Use uint32_t for locals count consistently in related testing functions. Change-Id: I5bb5a324c4f4ed1aafc37849f3762d7a9630da51 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2549966Reviewed-by:
-
Frank Emrich authored
This CL adds partial support for objects whose slow mode dictionaries are OrderedNameDictionaries. This is the case for all slow mode objects if V8_DICT_MODE_PROTOTYPES is enabled. Specifically, this CL contains bailouts to the runtime in places where the subsequent CSA or Torque code may have to access property dictionaries. These bailouts only happen if V8_DICT_MODE_PROTOTYPES is set, in which case the property dictionaries are of type OrderedNameDictionary, which cannot be handled by most CSA/Torque code. The idea is that these bailouts are temporary and will all be removed once we've implemented the actuall dictionary accesses in CSA/Torque. Bug: v8:7569, v8:11167 Change-Id: I8308b2f8ba2d4dbf7918da42f995ae270c825fff Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2545711 Commit-Queue: Frank Emrich <emrich@google.com> Reviewed-by:
Marja Hölttä <marja@chromium.org> Cr-Commit-Position: refs/heads/master@{#71311}
-
Dominik Inführ authored
Reset main_thread_local_isolate_ only after Heap::TearDown was executed. main_thread_local_isolate_ is still needed in there for e.g. HandleBase::IsDereferenceAllowed in MemoryMeasurement. Bug: chromium:1150867, v8:10315 Change-Id: Ia1ebfd561b7a3ab2d346f0c17b239f75ad77471f Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2549969Reviewed-by:
-
Michael Achenbach authored
This is a reland of e26863df The test now works with the no-i18n case. Original change's description: > [foozzie] Suppress access to CurrentTimeValue > > This stubs out CurrentTimeValue for differential fuzzing as otherwise > the non-deterministic value leaks from Intl.DateTimeFormat format and > formatToParts. > > This also affects other date creations, like Date.now(), which is > already stubbed out on the JS side. We keep that code for > backwards-compatibility to keep bisection stable. > > Bug: chromium:1149050 > Change-Id: Ifd82844c9fb8ce7262b55da6cf9f88f544268942 > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2546685 > Reviewed-by: Camillo Bruni <cbruni@chromium.org> > Reviewed-by: Clemens Backes <clemensb@chromium.org> > Commit-Queue: Michael Achenbach <machenbach@chromium.org> > Cr-Commit-Position: refs/heads/master@{#71294} Cq-Include-Trybots: luci.v8.try.triggered:v8_linux_noi18n_rel_ng_triggered Bug: chromium:1149050 Change-Id: I4a750b580495532ca0ffb125522f8f5958e4cad6 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2552401 Commit-Queue: Michael Achenbach <machenbach@chromium.org> Commit-Queue: Clemens Backes <clemensb@chromium.org> Reviewed-by:
Camillo Bruni <cbruni@chromium.org> Auto-Submit: Michael Achenbach <machenbach@chromium.org> Cr-Commit-Position: refs/heads/master@{#71309}
-
Clemens Backes authored
We had a test which first enabled the profiler, and then compiled wasm code. In this case, all code objects were registered correctly and the profile looked as expected. This CL extends the test for also test another order: First compile the wasm code, then enable the profiler. In that case, we were reporting a wrong debug name of the exported wasm function. The name of that function is spec'ed to be the string representation of the function index. But for debugging, we want to see a more meaningful name, identical to the name we show when reporting the code during compilation. This fix requires handlifying the {SharedFunctionInfo::DebugName} method, because for exported wasm functions, it needs to allocate a new name on the JS heap. In order to avoid this allocation where possible, a second variant is added which returns a unique_ptr directly. This can be used in all places where the name is just being printed, which turned out to be the majority of cases ({DebugName().ToCString()}). R=petermarshall@chromium.org Bug: chromium:1141787 Change-Id: I0343c2f06f0b852007535ff07459b712801ead01 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2543931 Commit-Queue: Clemens Backes <clemensb@chromium.org> Reviewed-by: -
Clemens Backes authored
This is a reland of 800307f6, with a minimal fix for arm64 (uint64_t -> uintptr_t). Original change's description: > [wasm][memory64] Prepare Liftoff for ptrsize offsets > > This CL prepares the LiftoffAssembler interface for uintptr_t offsets. > Many places can still only handle 32-bit values, but after this CL we can > start storing the offsets as uintptr_t in the memory access immediates. > Some TODOs are placed to extend code generation for 64-bit additions, if > memory64 is enabled. > All of this will be addressed in follow-up CLs. > > R=manoskouk@chromium.org > > Bug: v8:10949 > Change-Id: Id3b9b8aa555ab41f082ba012f4f8d80586c35b89 > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2529452 > Commit-Queue: Clemens Backes <clemensb@chromium.org> > Reviewed-by: Manos Koukoutos <manoskouk@chromium.org> > Cr-Commit-Position: refs/heads/master@{#71236} Bug: v8:10949 Tbr: manoskouk@chromium.org Change-Id: I33a9676afbf84d2032b181de2afd745841575900 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2550663Reviewed-by:
-
Mythri A authored
We added this to add elements kind to handlers so Turboprop could use the elements kind information for inlining array builtins when using dynamic map checks. This information isn't useful without other map based optimizations like constant folding to inline array builtins. So we don't need this information. Bug: v8:10582 Change-Id: I846c037ca2d87158dd017e2e23c7d1a0dfc685b4 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2549950 Commit-Queue: Mythri Alle <mythria@chromium.org> Reviewed-by:
Sathya Gunasekaran <gsathya@chromium.org> Cr-Commit-Position: refs/heads/master@{#71306}
-
Omer Katz authored
This reverts commit 64bf4c53. Reason for revert: fix in patchset 2 Original change's description: > Revert "cppgc: Use tracing scopes" > > This reverts commit 548fe208. > > Reason for revert: Issues on Mac64: https://ci.chromium.org/p/v8/builders/ci/V8%20Mac64%20-%20debug/31710 > > Original change's description: > > cppgc: Use tracing scopes > > > > The scopes themselves mostly have the same coverage as current scopes in > > blink. A few exception due to encapsulation exist and are highlighted as > > comments on the CL. > > > > Bug: chromium:1056170 > > Change-Id: I48af2cfdfd53a8caa1ab5d805d377f6f13a825bc > > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2540552 > > Commit-Queue: Omer Katz <omerkatz@chromium.org> > > Reviewed-by: Michael Lippautz <mlippautz@chromium.org> > > Reviewed-by: Ulan Degenbaev <ulan@chromium.org> > > Cr-Commit-Position: refs/heads/master@{#71285} > > TBR=ulan@chromium.org,mlippautz@chromium.org,omerkatz@chromium.org > > Change-Id: I20dce9309dcaeff6ea61bdc51df3a2f62c2a103f > No-Presubmit: true > No-Tree-Checks: true > No-Try: true > Bug: chromium:1056170 > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2550782 > Reviewed-by: Clemens Backes <clemensb@chromium.org> > Commit-Queue: Clemens Backes <clemensb@chromium.org> > Cr-Commit-Position: refs/heads/master@{#71289} TBR=ulan@chromium.org,mlippautz@chromium.org,clemensb@chromium.org,omerkatz@chromium.org # Not skipping CQ checks because this is a reland. Bug: chromium:1056170 Change-Id: I9c57cdb3b0d310366cdf3e6fbcd1254fdc363163 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2550666 Auto-Submit: Omer Katz <omerkatz@chromium.org> Reviewed-by:
Michael Lippautz <mlippautz@chromium.org> Commit-Queue: Michael Lippautz <mlippautz@chromium.org> Cr-Commit-Position: refs/heads/master@{#71305}
-
Frank Emrich authored
This CL adds partial support for objects whose slow mode dictionaries are OrderedNameDictionaries. This is the case for all slow mode objects if V8_DICT_MODE_PROTOTYPES is enabled. Specifically, this CL contains minor changes to CSA code, short of actually performing ordered dictionary lookups using CSA implementations of these lookups. Bug: v8:7569 Change-Id: I0dab0f21000ca3b9b170ace58787ec639d587e64 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2540590 Commit-Queue: Frank Emrich <emrich@google.com> Reviewed-by:
Dominik Inführ <dinfuehr@chromium.org> Cr-Commit-Position: refs/heads/master@{#71304}
-
