diff options
author | Vladimir Marko <vmarko@google.com> | 2020-07-17 14:35:56 +0100 |
---|---|---|
committer | Vladimir Marko <vmarko@google.com> | 2020-07-20 08:21:11 +0000 |
commit | 4307cd77319dd88eb65e88e5b4763250eff9c543 (patch) | |
tree | bec7345debfa105c1dd0e8c161eaf89281eb4d53 /tools/checker/file_format/c1visualizer/struct.py | |
parent | 76228ffa13c3101c1a74cc561c171c4064593796 (diff) |
LSE: Fix tracking heap values for small types.
We previously inserted TypeConversion to 8-bit and 16-bit
types only when replacing loads at the end of LSE. This is
insufficient as it allowed incorrect merging of values that
had different type. We now insert the TypeConversion when we
designate a load for replacement and therefore when a value
retrieved by such load is stored in another heap location,
we record the substitute TypeConversion as the heap value.
This replaces the insufficient fix from
https://android-review.googlesource.com/538635 .
Test: New tests in 530-checker-lse and 530-checker-lse3.
Test: m test-art-host-gtest
Test: testrunner.py --host --optimizing
Bug: 161521389
Change-Id: I7c41931126455411d25f0d675857f104700a15af
Diffstat (limited to 'tools/checker/file_format/c1visualizer/struct.py')
0 files changed, 0 insertions, 0 deletions