diff options
author | Nicolas Geoffray <ngeoffray@google.com> | 2015-04-03 11:02:38 +0100 |
---|---|---|
committer | Nicolas Geoffray <ngeoffray@google.com> | 2015-04-03 12:36:38 +0100 |
commit | 3dcd58cd54a922b864494fb7fff4a7f7a8562db9 (patch) | |
tree | 1e7b416de3dd46b0301f835632f8b3d0392beb97 /test/471-deopt-environment | |
parent | d43f160dc294655885a2c273307d34585c4ce97b (diff) | |
download | android_art-3dcd58cd54a922b864494fb7fff4a7f7a8562db9.tar.gz android_art-3dcd58cd54a922b864494fb7fff4a7f7a8562db9.tar.bz2 android_art-3dcd58cd54a922b864494fb7fff4a7f7a8562db9.zip |
Fix a bug when creating a HDeoptimization instruction.
We need to copy the environment, instead of just pointing
to an existing one. Otherwise, if the instruction that initially
holds the environemnt gets removed from the graph, any update
to an instruction in that environment will not be reflected in it.
bug:20058506
Change-Id: I2a62476d0851ecbc3707c0da395d8502ee437422
Diffstat (limited to 'test/471-deopt-environment')
-rw-r--r-- | test/471-deopt-environment/expected.txt | 0 | ||||
-rw-r--r-- | test/471-deopt-environment/info.txt | 3 | ||||
-rw-r--r-- | test/471-deopt-environment/src/Main.java | 47 |
3 files changed, 50 insertions, 0 deletions
diff --git a/test/471-deopt-environment/expected.txt b/test/471-deopt-environment/expected.txt new file mode 100644 index 0000000000..e69de29bb2 --- /dev/null +++ b/test/471-deopt-environment/expected.txt diff --git a/test/471-deopt-environment/info.txt b/test/471-deopt-environment/info.txt new file mode 100644 index 0000000000..bcb95754a4 --- /dev/null +++ b/test/471-deopt-environment/info.txt @@ -0,0 +1,3 @@ +Regression test for the bounds check elimination pass, which +uses to generate a HDeoptimization instruction with an +HEnvironment that does not have the uses lists updated. diff --git a/test/471-deopt-environment/src/Main.java b/test/471-deopt-environment/src/Main.java new file mode 100644 index 0000000000..5c5080be54 --- /dev/null +++ b/test/471-deopt-environment/src/Main.java @@ -0,0 +1,47 @@ +/* + * Copyright (C) 2015 The Android Open Source Project + * + * Licensed under the Apache License, Version 2.0 (the "License"); + * you may not use this file except in compliance with the License. + * You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + */ + +public class Main { + + private static int willInline(int a, int b) { + return a & b; + } + + static int[] a = new int[4]; + static int field = 42; + + public static void main(String[] args) throws Exception { + // The order of optimizations that would lead to the problem was: + // 1) Inlining of `willInline`. + // 2) Bounds check elimination inserting a deopt at a[0] and removing the HBoundsCheck. + // 3) Instruction simplifier simpilifying the inlined willInline to just `field`. + // + // At this point, if the environment of the HDeoptimization instruction was + // just a pointer to the one in a[0], the uses lists would have not been updated + // and the HBoundsCheck being dead code after the HDeoptimization, the simplifcation + // at step 3) would not updated that environment. + int inEnv = willInline(field, field); + int doAdds = a[0] + a[1] + a[2] + a[3]; + + if (inEnv != 42) { + throw new Error("Expected 42"); + } + + if (doAdds != 0) { + throw new Error("Expected 0"); + } + } +} |