Skip to content

Remove JvmIrDeserializer#5693

Open
ivandev0 wants to merge 4 commits intomasterfrom
rr/kylchik/drop-jvm-ir-deserialize
Open

Remove JvmIrDeserializer#5693
ivandev0 wants to merge 4 commits intomasterfrom
rr/kylchik/drop-jvm-ir-deserialize

Conversation

@ivandev0
Copy link
Contributor

This is a continuation of removing IR serialization feature from JVM. This time I removed IR deserialization.

@kotlin-safe-merge
Copy link

kotlin-safe-merge bot commented Feb 18, 2026

Code Owners

Rule Owners Approval
/​analysis/​ @JetBrains/kotlin-analysis-api
/​compiler/​cli/​, /​compiler/​fir/​, /​compiler/​tests-​common-​new/​, /​core/​compiler.​common.​jvm/​, /​core/​deserialization.​common.​jvm/​ @JetBrains/kotlin-frontend
/​compiler/​cli/​cli-​native-​klib/​ @JetBrains/kotlin-common-backend, @JetBrains/kotlin-frontend
/​compiler/​incremental-​compilation-​impl/​ @JetBrains/build-tools-api, @JetBrains/kotlin-build-tools
/​compiler/​ir/​backend.​jvm/​, /​compiler/​ir/​ir.​psi2ir/​, /​compiler/​ir/​serialization.​jvm/​ @JetBrains/kotlin-jvm
/​compiler/​ir/​ir.​tree/​ @JetBrains/kotlin-common-backend
/​compiler/​tests-​compiler-​utils/​ @JetBrains/kotlin-compiler
/​plugins/​compose/​ @ShikaSD, @andrewbailey, @bentrengrove, @chuckjaz, @derekxu16

@ivandev0 ivandev0 requested a review from udalov February 18, 2026 13:46
@ligee
Copy link
Contributor

ligee commented Feb 18, 2026

@ivandev0, it would be very helpful to place a reference to the reasons for removing IR (de)serialization into the commit messages, a link to the corresponding YT issue, or just a few words. I have no active context about it, so it's difficult to make a meaningful approval. But even if I did, such a reference would still benefit the future maintenance.

@ivandev0
Copy link
Contributor Author

@ivandev0, it would be very helpful to place a reference to the reasons for removing IR (de)serialization into the commit messages, a link to the corresponding YT issue, or just a few words. I have no active context about it, so it's difficult to make a meaningful approval. But even if I did, such a reference would still benefit the future maintenance.

Yes, sorry. I was a little bit lazy here. We don't have any tickets. Serialization of IR for JVM was an experimental feature for a long time, and the only known usage was at Google side. We finally got a green light to drop it completely.

As for the reason why we drop it, again, no tickets. The best thing I can give you is this comment.

Right now I am considering this as tech dept and have little to write into the commit message. I can create a ticket just for the sake of it, but I personally don't know the benefit of such a ticket.

@KotlinBuild KotlinBuild force-pushed the rr/kylchik/drop-jvm-ir-deserialize branch from 587b90e to e2f780d Compare February 19, 2026 10:12
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

Comments