diff options
author | Vladimir Marko <vmarko@google.com> | 2016-04-05 11:18:49 +0100 |
---|---|---|
committer | Vladimir Marko <vmarko@google.com> | 2016-05-10 10:26:11 +0100 |
commit | bf12e4d4209ac4e8fb98b4fd5193208adc7fe3ff (patch) | |
tree | 27ac87cda125c7182238e9ab80403b321690de1b /test/ProfileTestMultiDex | |
parent | ceeb3b73f84e8b11f362605007d382405d08d95c (diff) |
Optimizing: LoadString may not have any side effects.
LoadString does not have any side effects if the string is
known to be in the dex cache or it's a boot image string
referenced directly, as specified by the string load kind.
We need to clear the side effects for these cases to avoid
a DCHECK() failure when such LoadString instruction ends up
between a ClinitCheck and an instruction to which we want to
merge that ClinitCheck. This may happen as a consequence of
inlining, LICM and DCE as shown by a regression test.
Bug: 27929914
(cherry picked from commit ace7a000a433ce4ecf94f30adea39c01a76fa936)
Change-Id: Iaf9c63b6e58aae1e246b43ca52eea0b47a6ad565
Diffstat (limited to 'test/ProfileTestMultiDex')
0 files changed, 0 insertions, 0 deletions