[K/JS] Remove trivial JS constructors from JS AST

This commit is contained in:
Artem Kobzar
2023-07-19 10:41:04 +00:00
committed by Space Team
parent 4366164b65
commit 69c8942462
3 changed files with 31 additions and 17 deletions
@@ -20,7 +20,7 @@ import org.jetbrains.kotlin.js.backend.ast.JsClass
class ClassPostProcessor(val root: JsClass) {
val optimizations = listOf(
{ EmptyConstructorRemoval(root).apply() }
{ TrivialConstructorRemoval(root).apply() }
)
fun apply() {
@@ -1,16 +0,0 @@
/*
* Copyright 2010-2023 JetBrains s.r.o. and Kotlin Programming Language contributors.
* Use of this source code is governed by the Apache 2.0 license that can be found in the license/LICENSE.txt file.
*/
package org.jetbrains.kotlin.js.inline.clean
import org.jetbrains.kotlin.js.backend.ast.JsClass
class EmptyConstructorRemoval(private val klass: JsClass) {
fun apply(): Boolean {
if (klass.constructor?.body?.statements?.isEmpty() != true) return false
klass.constructor = null
return true
}
}
@@ -0,0 +1,30 @@
/*
* Copyright 2010-2023 JetBrains s.r.o. and Kotlin Programming Language contributors.
* Use of this source code is governed by the Apache 2.0 license that can be found in the license/LICENSE.txt file.
*/
package org.jetbrains.kotlin.js.inline.clean
import org.jetbrains.kotlin.js.backend.ast.JsClass
import org.jetbrains.kotlin.js.backend.ast.JsExpressionStatement
import org.jetbrains.kotlin.js.backend.ast.JsFunction
import org.jetbrains.kotlin.js.backend.ast.JsInvocation
import org.jetbrains.kotlin.js.backend.ast.JsNameRef
import org.jetbrains.kotlin.js.backend.ast.JsSuperRef
class TrivialConstructorRemoval(private val klass: JsClass) {
fun apply(): Boolean {
if (klass.constructor?.isTrivial() != true) return false
klass.constructor = null
return true
}
private fun JsFunction.isTrivial(): Boolean {
return body.statements.all { statement ->
val expressionStatement = statement as? JsExpressionStatement
val invocation = expressionStatement?.expression as? JsInvocation
invocation?.qualifier is JsSuperRef && invocation.arguments.size == parameters.size && invocation.arguments.withIndex()
.all { (index, argument) -> argument is JsNameRef && argument.name === parameters[index].name }
}
}
}