azalea/kotlin-fork
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

refactoring: extracted 'checkPseudocode'

Browse Source
This commit is contained in:
Svetlana Isakova
2014-03-05 14:44:29 +04:00
parent 5de4c5965c
commit 04947fa6a9
1 changed files with 17 additions and 19 deletions
Download Patch File Download Diff File Expand all files Collapse all files
compiler/tests/org/jetbrains/jet/cfg/AbstractControlFlowTest.java
+17 -19
View File
@@ -115,30 +115,28 @@ public abstract class AbstractControlFlowTest extends KotlinTestWithEnvironment
instructionDump.append("\n---------------------\n"); instructionDump.append("\n---------------------\n");
dumpInstructions((PseudocodeImpl) pseudocode, instructionDump); dumpInstructions((PseudocodeImpl) pseudocode, instructionDump);
instructionDump.append("=====================\n"); instructionDump.append("=====================\n");
checkPseudocode((PseudocodeImpl) pseudocode);
//check edges directions
Collection<Instruction> instructions = ((PseudocodeImpl)pseudocode).getAllInstructions();
for (Instruction instruction : instructions) {
if (!((InstructionImpl)instruction).isDead()) {
for (Instruction nextInstruction : instruction.getNextInstructions()) {
assertTrue("instruction '" + instruction + "' has '" + nextInstruction + "' among next instructions list, but not vice versa",
nextInstruction.getPreviousInstructions().contains(instruction));
}
for (Instruction prevInstruction : instruction.getPreviousInstructions()) {
assertTrue("instruction '" + instruction + "' has '" + prevInstruction + "' among previous instructions list, but not vice versa",
prevInstruction.getNextInstructions().contains(instruction));
}
}
}
} }
File expectedInstructionsFile = JetTestUtils.replaceExtension(file, "instructions"); File expectedInstructionsFile = JetTestUtils.replaceExtension(file, "instructions");
JetTestUtils.assertEqualsToFile(expectedInstructionsFile, instructionDump.toString()); JetTestUtils.assertEqualsToFile(expectedInstructionsFile, instructionDump.toString());
}
// StringBuilder graphDump = new StringBuilder(); private void checkPseudocode(PseudocodeImpl pseudocode) {
// for (Pseudocode pseudocode : pseudocodes) { //check edges directions
// topOrderDump(pseudocode.) Collection<Instruction> instructions = pseudocode.getAllInstructions();
// } for (Instruction instruction : instructions) {
if (!((InstructionImpl)instruction).isDead()) {
for (Instruction nextInstruction : instruction.getNextInstructions()) {
assertTrue("instruction '" + instruction + "' has '" + nextInstruction + "' among next instructions list, but not vice versa",
nextInstruction.getPreviousInstructions().contains(instruction));
}
for (Instruction prevInstruction : instruction.getPreviousInstructions()) {
assertTrue("instruction '" + instruction + "' has '" + prevInstruction + "' among previous instructions list, but not vice versa",
prevInstruction.getNextInstructions().contains(instruction));
}
}
}
} }
private static String formatInstruction(Instruction instruction, int maxLength, Set<Instruction> remainedAfterPostProcessInstructions) { private static String formatInstruction(Instruction instruction, int maxLength, Set<Instruction> remainedAfterPostProcessInstructions) {