blob: f985745e7a8cd5eca0c0df4999b735ae86dbe248 [file] [log] [blame]
David Brazdil46e2a392015-03-16 17:31:52 +00001/*
2 * Copyright (C) 2015 The Android Open Source Project
3 *
4 * Licensed under the Apache License, Version 2.0 (the "License");
5 * you may not use this file except in compliance with the License.
6 * You may obtain a copy of the License at
7 *
8 * http://www.apache.org/licenses/LICENSE-2.0
9 *
10 * Unless required by applicable law or agreed to in writing, software
11 * distributed under the License is distributed on an "AS IS" BASIS,
12 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
13 * See the License for the specific language governing permissions and
14 * limitations under the License.
15 */
16
17#include "boolean_simplifier.h"
18
19namespace art {
20
David Brazdil769c9e52015-04-27 13:54:09 +010021void HBooleanSimplifier::TryRemovingNegatedCondition(HBasicBlock* block) {
22 DCHECK(block->EndsWithIf());
23
24 // Check if the condition is a Boolean negation.
25 HIf* if_instruction = block->GetLastInstruction()->AsIf();
26 HInstruction* boolean_not = if_instruction->InputAt(0);
27 if (!boolean_not->IsBooleanNot()) {
28 return;
29 }
30
31 // Make BooleanNot's input the condition of the If and swap branches.
32 if_instruction->ReplaceInput(boolean_not->InputAt(0), 0);
33 block->SwapSuccessors();
34
35 // Remove the BooleanNot if it is now unused.
36 if (!boolean_not->HasUses()) {
37 boolean_not->GetBlock()->RemoveInstruction(boolean_not);
38 }
39}
40
David Brazdil46e2a392015-03-16 17:31:52 +000041// Returns true if 'block1' and 'block2' are empty, merge into the same single
42// successor and the successor can only be reached from them.
43static bool BlocksDoMergeTogether(HBasicBlock* block1, HBasicBlock* block2) {
44 if (!block1->IsSingleGoto() || !block2->IsSingleGoto()) return false;
Vladimir Markoec7802a2015-10-01 20:57:57 +010045 HBasicBlock* succ1 = block1->GetSuccessors()[0];
46 HBasicBlock* succ2 = block2->GetSuccessors()[0];
Vladimir Marko60584552015-09-03 13:35:12 +000047 return succ1 == succ2 && succ1->GetPredecessors().size() == 2u;
David Brazdil46e2a392015-03-16 17:31:52 +000048}
49
50// Returns true if the outcome of the branching matches the boolean value of
51// the branching condition.
52static bool PreservesCondition(HInstruction* input_true, HInstruction* input_false) {
David Brazdilb2bd1c52015-03-25 11:17:37 +000053 return input_true->IsIntConstant() && input_true->AsIntConstant()->IsOne()
54 && input_false->IsIntConstant() && input_false->AsIntConstant()->IsZero();
David Brazdil46e2a392015-03-16 17:31:52 +000055}
56
57// Returns true if the outcome of the branching is exactly opposite of the
58// boolean value of the branching condition.
59static bool NegatesCondition(HInstruction* input_true, HInstruction* input_false) {
David Brazdilb2bd1c52015-03-25 11:17:37 +000060 return input_true->IsIntConstant() && input_true->AsIntConstant()->IsZero()
61 && input_false->IsIntConstant() && input_false->AsIntConstant()->IsOne();
David Brazdil46e2a392015-03-16 17:31:52 +000062}
63
64// Returns an instruction with the opposite boolean value from 'cond'.
65static HInstruction* GetOppositeCondition(HInstruction* cond) {
66 HGraph* graph = cond->GetBlock()->GetGraph();
67 ArenaAllocator* allocator = graph->GetArena();
68
69 if (cond->IsCondition()) {
70 HInstruction* lhs = cond->InputAt(0);
71 HInstruction* rhs = cond->InputAt(1);
Aart Bike9f37602015-10-09 11:15:55 -070072 switch (cond->AsCondition()->GetOppositeCondition()) { // get *opposite*
73 case kCondEQ: return new (allocator) HEqual(lhs, rhs);
74 case kCondNE: return new (allocator) HNotEqual(lhs, rhs);
75 case kCondLT: return new (allocator) HLessThan(lhs, rhs);
76 case kCondLE: return new (allocator) HLessThanOrEqual(lhs, rhs);
77 case kCondGT: return new (allocator) HGreaterThan(lhs, rhs);
78 case kCondGE: return new (allocator) HGreaterThanOrEqual(lhs, rhs);
79 case kCondB: return new (allocator) HBelow(lhs, rhs);
80 case kCondBE: return new (allocator) HBelowOrEqual(lhs, rhs);
81 case kCondA: return new (allocator) HAbove(lhs, rhs);
82 case kCondAE: return new (allocator) HAboveOrEqual(lhs, rhs);
David Brazdil46e2a392015-03-16 17:31:52 +000083 }
84 } else if (cond->IsIntConstant()) {
David Brazdilb2bd1c52015-03-25 11:17:37 +000085 HIntConstant* int_const = cond->AsIntConstant();
86 if (int_const->IsZero()) {
David Brazdil8d5b8b22015-03-24 10:51:52 +000087 return graph->GetIntConstant(1);
David Brazdil46e2a392015-03-16 17:31:52 +000088 } else {
David Brazdilb2bd1c52015-03-25 11:17:37 +000089 DCHECK(int_const->IsOne());
David Brazdil8d5b8b22015-03-24 10:51:52 +000090 return graph->GetIntConstant(0);
David Brazdil46e2a392015-03-16 17:31:52 +000091 }
92 }
Aart Bike9f37602015-10-09 11:15:55 -070093 // General case when 'cond' is another instruction of type boolean,
94 // as verified by SSAChecker.
95 return new (allocator) HBooleanNot(cond);
David Brazdil46e2a392015-03-16 17:31:52 +000096}
97
David Brazdil769c9e52015-04-27 13:54:09 +010098void HBooleanSimplifier::TryRemovingBooleanSelection(HBasicBlock* block) {
99 DCHECK(block->EndsWithIf());
100
101 // Find elements of the pattern.
102 HIf* if_instruction = block->GetLastInstruction()->AsIf();
103 HBasicBlock* true_block = if_instruction->IfTrueSuccessor();
104 HBasicBlock* false_block = if_instruction->IfFalseSuccessor();
105 if (!BlocksDoMergeTogether(true_block, false_block)) {
106 return;
107 }
Vladimir Markoec7802a2015-10-01 20:57:57 +0100108 HBasicBlock* merge_block = true_block->GetSuccessors()[0];
David Brazdil769c9e52015-04-27 13:54:09 +0100109 if (!merge_block->HasSinglePhi()) {
110 return;
111 }
112 HPhi* phi = merge_block->GetFirstPhi()->AsPhi();
113 HInstruction* true_value = phi->InputAt(merge_block->GetPredecessorIndexOf(true_block));
114 HInstruction* false_value = phi->InputAt(merge_block->GetPredecessorIndexOf(false_block));
115
116 // Check if the selection negates/preserves the value of the condition and
117 // if so, generate a suitable replacement instruction.
118 HInstruction* if_condition = if_instruction->InputAt(0);
Mark Mendellc4701932015-04-10 13:18:51 -0400119
120 // Don't change FP compares. The definition of compares involving NaNs forces
121 // the compares to be done as written by the user.
122 if (if_condition->IsCondition() &&
123 Primitive::IsFloatingPointType(if_condition->InputAt(0)->GetType())) {
124 return;
125 }
126
David Brazdil769c9e52015-04-27 13:54:09 +0100127 HInstruction* replacement;
128 if (NegatesCondition(true_value, false_value)) {
129 replacement = GetOppositeCondition(if_condition);
130 if (replacement->GetBlock() == nullptr) {
131 block->InsertInstructionBefore(replacement, if_instruction);
132 }
133 } else if (PreservesCondition(true_value, false_value)) {
134 replacement = if_condition;
135 } else {
136 return;
137 }
138
139 // Replace the selection outcome with the new instruction.
140 phi->ReplaceWith(replacement);
141 merge_block->RemovePhi(phi);
142
143 // Delete the true branch and merge the resulting chain of blocks
144 // 'block->false_block->merge_block' into one.
145 true_block->DisconnectAndDelete();
146 block->MergeWith(false_block);
147 block->MergeWith(merge_block);
148
Nicolas Geoffray1f82ecc2015-06-24 12:20:24 +0100149 // No need to update any dominance information, as we are simplifying
150 // a simple diamond shape, where the join block is merged with the
151 // entry block. Any following blocks would have had the join block
152 // as a dominator, and `MergeWith` handles changing that to the
153 // entry block.
David Brazdil769c9e52015-04-27 13:54:09 +0100154}
155
David Brazdil46e2a392015-03-16 17:31:52 +0000156void HBooleanSimplifier::Run() {
157 // Iterate in post order in the unlikely case that removing one occurrence of
David Brazdil769c9e52015-04-27 13:54:09 +0100158 // the selection pattern empties a branch block of another occurrence.
159 // Otherwise the order does not matter.
David Brazdil46e2a392015-03-16 17:31:52 +0000160 for (HPostOrderIterator it(*graph_); !it.Done(); it.Advance()) {
161 HBasicBlock* block = it.Current();
David Brazdilb2bd1c52015-03-25 11:17:37 +0000162 if (!block->EndsWithIf()) continue;
David Brazdil46e2a392015-03-16 17:31:52 +0000163
David Brazdil769c9e52015-04-27 13:54:09 +0100164 // If condition is negated, remove the negation and swap the branches.
165 TryRemovingNegatedCondition(block);
David Brazdil46e2a392015-03-16 17:31:52 +0000166
David Brazdil769c9e52015-04-27 13:54:09 +0100167 // If this is a boolean-selection diamond pattern, replace its result with
168 // the condition value (or its negation) and simplify the graph.
169 TryRemovingBooleanSelection(block);
David Brazdil46e2a392015-03-16 17:31:52 +0000170 }
171}
172
173} // namespace art