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

Debugger, BE: Make the NOP instruction optimizer save the first NOP, not the last

Browse Source 
This commit fixes the following tests:
 - KotlinSteppingTestGenerated.StepOver#testSoInlineLibFun
 - KotlinSteppingTestGenerated.StepOver#testSoInlineIterableFun
 - KotlinSteppingTestGenerated.StepOver#testSoInlineFunOnOneLineFor
This commit is contained in:
Yan Zhulanow
2019-11-28 03:46:42 +09:00
parent 9cba72d5dd
commit 8b17718086
2 changed files with 5 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
compiler/backend/src/org/jetbrains/kotlin/codegen/optimization/RedundantNopsCleanupMethodTransformer.kt
+4 -4
View File
@@ -103,16 +103,16 @@ class RedundantNopsCleanupMethodTransformer : MethodTransformer() {
internal fun getRequiredNopInRange(firstInclusive: AbstractInsnNode, lastExclusive: AbstractInsnNode?): AbstractInsnNode? {
var lastNop: AbstractInsnNode? = null
var firstNop: AbstractInsnNode? = null
var current: AbstractInsnNode? = firstInclusive
while (current != null && current != lastExclusive) {
if (current.isMeaningful && current.opcode != Opcodes.NOP) {
return null
} else if (current.opcode == Opcodes.NOP) {
lastNop = current
} else if (current.opcode == Opcodes.NOP && firstNop == null) {
firstNop = current
}
current = current.next
}
return lastNop
return firstNop
}