• Ben Smith's avatar
    Remove volatile from TypedArray set/get · 695cc8fe
    Ben Smith authored
    It causes a significant regression in TypedArray benchmarks. Thinking it
    through, it is actually not necessary for the JavaScript memory model
    either. I originally added it to ensure that reads/writes are not elided
    or duplicated, but there is no guarantee of this behavior for non-atomic
    writes in the model.
    
    BUG=chromium:763814
    R=clemensh@chromium.org
    
    Change-Id: Ib03d2e2e77a846d4b9e84eebc7f8fbf861f8fd7c
    Reviewed-on: https://chromium-review.googlesource.com/661192Reviewed-by: 's avatarClemens Hammacher <clemensh@chromium.org>
    Commit-Queue: Ben Smith <binji@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#47957}
    695cc8fe
Name
Last commit
Last update
benchmarks Loading commit data...
build_overrides Loading commit data...
docs Loading commit data...
gni Loading commit data...
gypfiles Loading commit data...
include Loading commit data...
infra Loading commit data...
samples Loading commit data...
src Loading commit data...
test Loading commit data...
testing Loading commit data...
third_party Loading commit data...
tools Loading commit data...
.clang-format Loading commit data...
.editorconfig Loading commit data...
.gitignore Loading commit data...
.gn Loading commit data...
.ycm_extra_conf.py Loading commit data...
AUTHORS Loading commit data...
BUILD.gn Loading commit data...
CODE_OF_CONDUCT.md Loading commit data...
ChangeLog Loading commit data...
DEPS Loading commit data...
LICENSE Loading commit data...
LICENSE.fdlibm Loading commit data...
LICENSE.strongtalk Loading commit data...
LICENSE.v8 Loading commit data...
LICENSE.valgrind Loading commit data...
Makefile Loading commit data...
Makefile.android Loading commit data...
OWNERS Loading commit data...
PRESUBMIT.py Loading commit data...
README.md Loading commit data...
WATCHLISTS Loading commit data...
codereview.settings Loading commit data...
snapshot_toolchain.gni Loading commit data...