diff options
| author | Vladimir Marko <vmarko@google.com> | 2019-05-16 12:50:47 +0000 |
|---|---|---|
| committer | Vladimir Marko <vmarko@google.com> | 2019-05-23 11:47:28 +0000 |
| commit | 8fa839cfe6f72adabdf79f938c57300e589e0803 (patch) | |
| tree | b940832441ef1b0c724c1383228225845e5d72d1 /test/521-checker-array-set-null/src/Main.java | |
| parent | 3cddf4538a8df429a6084f7176c07d55e1e0ec67 (diff) | |
Revert^2 "Improve ArraySet codegen."
This reverts commit 0dda8c84938d6bb4ce5a1707e5e109ea187fc33d.
The original change had two issues that have been fixed.
First, for heap poisoning, the null branch skipped over the
reference poisoning instructions which copy and poison the
value, thus writing whatever was left in the register.
Second, the change erroneously assumed that the slow path
performed only the assignability check and bound the slow
path exit label before the actual array store, unnecessarily
re-doing the store.
Change-Id: I9f380efa12aa807b4f566a932dbc9dae824fb25a
Test: m test-art-host-gtest
Test: testrunner.py --host --optimizing
Test: aosp_taimen-userdebug boots.
Test: testrunner.py --target --optimizing
Test: Repeat the above with
ART_USE_READ_BARRIER=false ART_HEAP_POISONING=true
Bug: 32489401
Diffstat (limited to 'test/521-checker-array-set-null/src/Main.java')
0 files changed, 0 insertions, 0 deletions