Sancaros/phantasmal-world
1
0
Fork 0
mirror of https://github.com/DaanVandenBosch/phantasmal-world.git synced 2025-04-04 22:58:29 +08:00
Code Issues Actions Packages Projects Releases Wiki Activity

switch_jmp is a conditional jump after all.

Browse Source
This commit is contained in:
Daan Vanden Bosch 2020-11-15 00:32:33 +01:00
parent cd70d22da2
commit 4e65cc1882
2 changed files with 4 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
lib/src/commonMain/kotlin/world/phantasmal/lib/assembly/dataFlowAnalysis/ControlFlowGraph.kt
View File

@ -11,13 +11,12 @@ enum class BranchType {
None,
/**
* ret
* ret.
*/
Return,
/**
* jmp or switch_jmp. switch_jmp is a non-conditional jump because it always jumps even though
* the jump location is dynamic.
* jmp.
*/
Jump,
@ -200,7 +199,7 @@ private fun createBasicBlocks(cfg: ControlFlowGraphBuilder, segment: Instruction
branchLabels = listOf(inst.args[2].value as Int)
}
OP_SWITCH_JMP.code -> {
branchType = BranchType.Jump
branchType = BranchType.ConditionalJump
branchLabels = inst.args.drop(1).map { it.value as Int }
}

2
lib/src/commonMain/kotlin/world/phantasmal/lib/fileFormats/quest/ByteCode.kt
View File

@ -411,7 +411,7 @@ private fun parseInstructionsSegment(
for (i in instructions.size - 1 downTo 0) {
val opcode = instructions[i].opcode.code
if (opcode == OP_RET.code || opcode == OP_JMP.code || opcode == OP_SWITCH_JMP.code) {
if (opcode == OP_RET.code || opcode == OP_JMP.code) {
dropThrough = false
break
}