VV8 LUCI CQ[zone] Clean up ZoneAllocator
| 文件 | 最后提交记录 | 最后更新时间 |
|---|---|---|
[zone] Clean up ZoneAllocator This CL cleans up the implementations of ZoneAllocator and RecyclingZoneAllocator, removing redundant or deprecated things and aligning with C++ >= 11. It also adds tests to verify that both these allocators can be used with standard C++ containers. Change-Id: I245b39d2bd39f7fbbf7b5a589a973701314d9861 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/4866420 Reviewed-by: Anton Bikineev <bikineev@chromium.org> Reviewed-by: Michael Lippautz <mlippautz@chromium.org> Commit-Queue: Nikolaos Papaspyrou <nikolaos@chromium.org> Cr-Commit-Position: refs/heads/main@{#90007} | 2 年前 | |
[zone] Rename Zone::NewArray to Zone::AllocateArray The function does not construct the objects in the array and therefore is not comparable to new T[length] in C++. Follow-up change for renaming NewVector: https://crrev.com/c/4685550 Change-Id: I0db92dae2895bbb1162a3f7a483025ca89adf819 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/4686973 Reviewed-by: Tobias Tebbi <tebbi@chromium.org> Commit-Queue: Matthias Liedtke <mliedtke@chromium.org> Cr-Commit-Position: refs/heads/main@{#88985} | 2 年前 | |
[zone] ZoneHandleSet -> ZoneCompactSet Refactor ZoneHandleSet to use a traits class, to allow Handle-like uses which don't use actual Handles (the intended use case is compiler heap refs, in a followup). Since this doesn't deal with Handles anymore, rename to ZoneCompactSet (since this is the intended usage). Additionally, do some refactoring to improve efficiency where possible, in particular using lower_bound/binary_search to take advantage of the sorted list (which was already maintained sorted but this order wasn't used), use base::Vector instead of ZoneVector since we're copying things anyway, and wrap up the tagging logic in an actual PointerWithPayload. Change-Id: Ia228fbf2fe0b3f8a5f78e7671d11de6cbb5c4ea1 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/4384478 Commit-Queue: Leszek Swirski <leszeks@chromium.org> Reviewed-by: Jakob Linke <jgruber@chromium.org> Reviewed-by: Clemens Backes <clemensb@chromium.org> Auto-Submit: Leszek Swirski <leszeks@chromium.org> Cr-Commit-Position: refs/heads/main@{#86799} | 3 年前 | |
[test] Add a unittest platform setup mixin Change the unittest runner to no longer uncondtionally set up a default platform in the "environment", but to instead make platform set-up part of the "mixin" framework for test fixtures. Requires modifying some tests that expect the platform to be available, and all flag implications resolved, before the mixin constructors run. We still keep the environment for setting up the process for cppgc. This process setup can only be done once per process, so it can no longer use the platform -- that's ok though, the page allocator used by cppgc's process initialisation doesn't have to be the same as the platform's so we can just pass in a separate new one. Change-Id: Ic8ccf39722e8212962c5bba87350c4b304388a7c Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3571886 Reviewed-by: Michael Lippautz <mlippautz@chromium.org> Auto-Submit: Leszek Swirski <leszeks@chromium.org> Commit-Queue: Leszek Swirski <leszeks@chromium.org> Cr-Commit-Position: refs/heads/main@{#79820} | 4 年前 | |
[test] Fix MSVC compilation of ZoneVectorTest Change-Id: I9904aaf975624be24f02929cfe8c26f025ec2819 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/4264287 Reviewed-by: Manos Koukoutos <manoskouk@chromium.org> Commit-Queue: Jakob Kummerow <jkummerow@chromium.org> Auto-Submit: Jakob Kummerow <jkummerow@chromium.org> Cr-Commit-Position: refs/heads/main@{#85891} | 3 年前 |