summaryrefslogtreecommitdiff
path: root/compiler/dex/mir_optimization.cc
diff options
context:
space:
mode:
author Vladimir Marko <vmarko@google.com> 2014-10-13 22:09:51 +0000
committer Android Git Automerger <android-git-automerger@android.com> 2014-10-13 22:09:51 +0000
commit61357e47ca5381621d73abf3d5683cfd6af4fa78 (patch)
tree1605cf8308ccf864040e1727fda3de6c12bf5ea9 /compiler/dex/mir_optimization.cc
parentcc4f8f96227b7d2065103ab775e7e7c9a762e624 (diff)
parentc39eb4aef19b7fe64283fb3ae7ce6d13cd6c8294 (diff)
am c39eb4ae: Merge "Add regression test for null check elimination."
* commit 'c39eb4aef19b7fe64283fb3ae7ce6d13cd6c8294': Add regression test for null check elimination.
Diffstat (limited to 'compiler/dex/mir_optimization.cc')
-rw-r--r--compiler/dex/mir_optimization.cc2
1 files changed, 2 insertions, 0 deletions
diff --git a/compiler/dex/mir_optimization.cc b/compiler/dex/mir_optimization.cc
index 2ac0b51a89..84c056daf3 100644
--- a/compiler/dex/mir_optimization.cc
+++ b/compiler/dex/mir_optimization.cc
@@ -929,6 +929,8 @@ bool MIRGraph::EliminateNullChecks(BasicBlock* bb) {
mir->optimization_flags |= MIR_IGNORE_NULL_CHECK;
} else {
// Do the null check.
+ // TODO: Rewrite the pass to converge first before doing any modifications so that
+ // we don't lose the MIR_IGNORE_NULL_CHECK here if previously set by some other pass.
mir->optimization_flags &= ~MIR_IGNORE_NULL_CHECK;
// Mark s_reg as null-checked
ssa_regs_to_check->ClearBit(src_sreg);