• Leszek Swirski's avatar
    Revert "[turbofan] Further optimize DataView accesses." · 6a62d88e
    Leszek Swirski authored
    This reverts commit c46915b9.
    
    Reason for revert: Disasm failures https://ci.chromium.org/p/v8/builders/luci.v8.ci/V8%20Linux%20-%20debug/21727 
    
    Original change's description:
    > [turbofan] Further optimize DataView accesses.
    > 
    > This adds support for unaligned load/store access to the DataView
    > backing store and uses byteswap operations to fix up the endianess
    > when necessary. This changes the Word32ReverseBytes operator to be
    > a required operator and adds the missing support on the Intel and
    > ARM platforms (on 64-bit platforms the Word64ReverseBytes operator
    > is also mandatory now).
    > 
    > This further improves the performance on the dataviewperf.js test
    > mentioned in the tracking bug by up to 40%, and at the same time
    > reduces the code complexity in the EffectControlLinearizer.
    > 
    > Bug: chromium:225811
    > Change-Id: I296170b828c2ccc1c317ed37840b564aa14cdec2
    > Reviewed-on: https://chromium-review.googlesource.com/1172777
    > Commit-Queue: Benedikt Meurer <bmeurer@chromium.org>
    > Reviewed-by: Sigurd Schneider <sigurds@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#55099}
    
    TBR=sigurds@chromium.org,bmeurer@chromium.org
    
    Change-Id: If7a62e3a1a4ad26823fcbd2ab6eb4c053ad11c49
    No-Presubmit: true
    No-Tree-Checks: true
    No-Try: true
    Bug: chromium:225811
    Reviewed-on: https://chromium-review.googlesource.com/1174171Reviewed-by: 's avatarLeszek Swirski <leszeks@chromium.org>
    Commit-Queue: Leszek Swirski <leszeks@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#55107}
    6a62d88e
assembler-arm.h 66.6 KB