• Michael Achenbach's avatar
    Revert "[es2015] Introduce JSDataView::external_pointer." · ec216398
    Michael Achenbach authored
    This reverts commit 46573e51.
    
    Reason for revert: Speculative revert for breaking chromium integration.
    
    Might break gpu tests and linux debug:
    https://ci.chromium.org/p/v8/builders/luci.v8.ci/Mac%20V8%20FYI%20Release%20(Intel)/2554
    
    Also blocks the roll:
    https://chromium-review.googlesource.com/c/chromium/src/+/1234328
    
    Original change's description:
    > [es2015] Introduce JSDataView::external_pointer.
    > 
    > This adds a new external_pointer field to every JSDataView instance
    > which points directly into the backing store at the given view's
    > byte_offset. This was the DataView performance is now almost on
    > par with the TypedArray performance for accessing aligned memory
    > (with appropriate endianess). This also serves as prepatory work
    > to enable full 64-bit addressing of DataView backing stores in
    > optimized code (soonish).
    > 
    > This change optimizes the bounds checking sequence in TurboFan in
    > such a way that it further improves the DataView set/get performance
    > by around 10%, almost closing the remaining gap between DataViews
    > and TypedArrays.
    > 
    > Drive-by-fix: Get rid of the code duplication around DataView inlining
    > in the JSCallReducer and have only a single bottleneck method now.
    > 
    > Bug: chromium:225811, v8:4153, v8:7881, v8:8171
    > Change-Id: I9118efd4d19e93f0e51c931a9bec1a56a0f4593e
    > Reviewed-on: https://chromium-review.googlesource.com/1231994
    > Commit-Queue: Benedikt Meurer <bmeurer@chromium.org>
    > Reviewed-by: Benedikt Meurer <bmeurer@chromium.org>
    > Reviewed-by: Tobias Tebbi <tebbi@chromium.org>
    > Reviewed-by: Yang Guo <yangguo@chromium.org>
    > Reviewed-by: Michael Lippautz <mlippautz@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#56042}
    
    TBR=yangguo@chromium.org,mlippautz@chromium.org,tebbi@chromium.org,bmeurer@chromium.org
    
    Change-Id: I614a90043b1574b19936c37987db94806cac3bd7
    No-Presubmit: true
    No-Tree-Checks: true
    No-Try: true
    Bug: chromium:225811, v8:4153, v8:7881, v8:8171
    Reviewed-on: https://chromium-review.googlesource.com/1234417Reviewed-by: 's avatarMichael Achenbach <machenbach@chromium.org>
    Commit-Queue: Michael Achenbach <machenbach@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#56059}
    ec216398
Name
Last commit
Last update
..
OWNERS Loading commit data...
array-buffer-collector.cc Loading commit data...
array-buffer-collector.h Loading commit data...
array-buffer-tracker-inl.h Loading commit data...
array-buffer-tracker.cc Loading commit data...
array-buffer-tracker.h Loading commit data...
barrier.h Loading commit data...
code-stats.cc Loading commit data...
code-stats.h Loading commit data...
concurrent-marking.cc Loading commit data...
concurrent-marking.h Loading commit data...
embedder-tracing.cc Loading commit data...
embedder-tracing.h Loading commit data...
factory-inl.h Loading commit data...
factory.cc Loading commit data...
factory.h Loading commit data...
gc-idle-time-handler.cc Loading commit data...
gc-idle-time-handler.h Loading commit data...
gc-tracer.cc Loading commit data...
gc-tracer.h Loading commit data...
heap-controller.cc Loading commit data...
heap-controller.h Loading commit data...
heap-inl.h Loading commit data...
heap-write-barrier-inl.h Loading commit data...
heap-write-barrier.h Loading commit data...
heap.cc Loading commit data...
heap.h Loading commit data...
incremental-marking-inl.h Loading commit data...
incremental-marking-job.cc Loading commit data...
incremental-marking-job.h Loading commit data...
incremental-marking.cc Loading commit data...
incremental-marking.h Loading commit data...
invalidated-slots-inl.h Loading commit data...
invalidated-slots.cc Loading commit data...
invalidated-slots.h Loading commit data...
item-parallel-job.cc Loading commit data...
item-parallel-job.h Loading commit data...
local-allocator-inl.h Loading commit data...
local-allocator.h Loading commit data...
mark-compact-inl.h Loading commit data...
mark-compact.cc Loading commit data...
mark-compact.h Loading commit data...
marking.cc Loading commit data...
marking.h Loading commit data...
memory-reducer.cc Loading commit data...
memory-reducer.h Loading commit data...
object-stats.cc Loading commit data...
object-stats.h Loading commit data...
objects-visiting-inl.h Loading commit data...
objects-visiting.cc Loading commit data...
objects-visiting.h Loading commit data...
remembered-set.h Loading commit data...
scavenge-job.cc Loading commit data...
scavenge-job.h Loading commit data...
scavenger-inl.h Loading commit data...
scavenger.cc Loading commit data...
scavenger.h Loading commit data...
setup-heap-internal.cc Loading commit data...
slot-set.h Loading commit data...
spaces-inl.h Loading commit data...
spaces.cc Loading commit data...
spaces.h Loading commit data...
store-buffer-inl.h Loading commit data...
store-buffer.cc Loading commit data...
store-buffer.h Loading commit data...
stress-marking-observer.cc Loading commit data...
stress-marking-observer.h Loading commit data...
stress-scavenge-observer.cc Loading commit data...
stress-scavenge-observer.h Loading commit data...
sweeper.cc Loading commit data...
sweeper.h Loading commit data...
worklist.h Loading commit data...