Skip to content

Add IC for Compose Wasm #809

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 32 commits into
base: master
Choose a base branch
from
Open

Add IC for Compose Wasm #809

wants to merge 32 commits into from

Conversation

nikpachoo
Copy link
Collaborator

Copy of #775, created by @ilgonmic.

@dkrasnoff
Copy link
Collaborator

dkrasnoff commented Jul 11, 2025

We are not merging this Pull Request for two reasons:

  1. The changes from the JS side are not merged in the Kotlin master (see we use the dev Kotlin version in this branch)
  2. There was a performance drawdown for cold start compared with the branch alpha-compose-stage

UPD(17.07.2025) - only 1st left

@dkrasnoff dkrasnoff force-pushed the compose-deployment branch from 52e4b0a to 4e52b2e Compare July 17, 2025 15:56
@dkrasnoff dkrasnoff marked this pull request as ready for review July 17, 2025 15:56
@dkrasnoff dkrasnoff self-requested a review as a code owner July 17, 2025 15:56
@@ -0,0 +1,10 @@
import org.gradle.api.attributes.Attribute

enum class CacheAttribute {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

With @ilgonmic we've discussed left only FULL.
Thus, the whole attribute can be deleted.

@@ -39,6 +40,7 @@ kotlin-compiler-ide = { group = "org.jetbrains.kotlin", name = "kotlin-compiler-
kotlin-idea = { group = "org.jetbrains.kotlin", name = "idea", version.ref = "kotlinIdeVersionWithSuffix" }
kotlin-core = { group = "org.jetbrains.kotlin", name = "core", version.ref = "kotlinIdeVersionWithSuffix" }
kotlinx-coroutines-core-jvm = { group = "org.jetbrains.kotlinx", name = "kotlinx-coroutines-core-jvm", version.ref = "kotlinx-coroutines" }
kotlinx-coroutines-core-compose-wasm = { group = "org.jetbrains.kotlinx", name = "kotlinx-coroutines-core-wasm-js", version.ref = "kotlinx-coroutines-compose-wasm" }
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can it be just a kotlinx-coroutines-core dependency?

.map { it.file("stdlib_master.wasm") }

rootProject.plugins.withType<org.jetbrains.kotlin.gradle.targets.js.binaryen.BinaryenRootPlugin> {
rootProject.the<BinaryenRootEnvSpec>().version = "122"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why do we need such hardcoding?

@@ -0,0 +1,97 @@
@file:OptIn(ExperimentalWasmDsl::class)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ilgonmic could you also add a Readme file to this new module?

@@ -0,0 +1,155 @@
import org.gradle.kotlin.dsl.support.serviceOf
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ilgonmic could you also add a Readme file to this new module?

compilerOptions.freeCompilerArgs.add("-Xwasm-multimodule-mode=master")
}

(binary as WasmBinary).optimizeTask.configure {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ilgonmic when should it work?

@@ -1,5 +1,5 @@
[versions]
kotlin = "2.1.0"
kotlin = "2.2.0-dev-8507"
Copy link
Collaborator

@dkrasnoff dkrasnoff Jul 10, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(Note for me, just not to forget) We can not change this version, because there are no changes in master branch

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants