Skip to content

Conversation

@ilgonmic
Copy link
Collaborator

It adds support of compiler backend incremental compilation for Kotlin JS IR.
It makes compilation much faster than before.

It makes some prepared cache, stored it and then on request, it copied to temp directory and using during compilation process.

Additionally compressing on server is enabled to make sending scripts less

@AlexanderPrendota
Copy link
Collaborator

@ilgonmic hi, is it possible to add tests?

@ilgonmic
Copy link
Collaborator Author

ilgonmic commented Feb 13, 2023

Tests on JS IR already exists
https://github.com/AlexanderPrendota/kotlin-compiler-server/blob/master/src/test/kotlin/com/compiler/server/ConvertToJsRunnerTest.kt#L36
I changed implementation details, that it should use incremental cache.

Or you mean some specific tests that incrementality work? I can think how to check it

@AlexanderPrendota AlexanderPrendota merged commit ce993ed into JetBrains:1.8.20-Beta Feb 14, 2023
@ilgonmic ilgonmic deleted the ilgonmic/js-ir-ic branch February 23, 2023 15:43
woainikk pushed a commit that referenced this pull request Feb 27, 2023
* Enable gzip

* process source module

* Support IC caches

* Init IC caches

* Prepare IC caches and copy it inside a request
woainikk pushed a commit that referenced this pull request Feb 27, 2023
* Enable gzip

* process source module

* Support IC caches

* Init IC caches

* Prepare IC caches and copy it inside a request
woainikk pushed a commit that referenced this pull request Mar 1, 2023
* Enable gzip

* process source module

* Support IC caches

* Init IC caches

* Prepare IC caches and copy it inside a request
woainikk pushed a commit that referenced this pull request Mar 8, 2023
* Enable gzip

* process source module

* Support IC caches

* Init IC caches

* Prepare IC caches and copy it inside a request
ilgonmic added a commit to ilgonmic/kotlin-compiler-server that referenced this pull request Mar 14, 2023
ilgonmic added a commit to ilgonmic/kotlin-compiler-server that referenced this pull request Mar 16, 2023
woainikk pushed a commit that referenced this pull request Mar 20, 2023
* Enable gzip

* process source module

* Support IC caches

* Init IC caches

* Prepare IC caches and copy it inside a request
woainikk pushed a commit that referenced this pull request Mar 20, 2023
woainikk pushed a commit that referenced this pull request Mar 23, 2023
* Enable gzip

* process source module

* Support IC caches

* Init IC caches

* Prepare IC caches and copy it inside a request
woainikk pushed a commit that referenced this pull request Mar 23, 2023
woainikk pushed a commit that referenced this pull request Mar 30, 2023
* Enable gzip

* process source module

* Support IC caches

* Init IC caches

* Prepare IC caches and copy it inside a request
woainikk pushed a commit that referenced this pull request Mar 30, 2023
AlexanderPrendota pushed a commit that referenced this pull request Apr 3, 2023
* chore(release): add kotlin 1.8.20-Beta

* Adopt changes from 1.8.20-Beta with private fields

* Infer JDK_HOME from java.home if it wasn't passed explicitly

 Now it requires because of KT-54337, where initialization of JDK_HOME was removed: JetBrains/kotlin@b50a803#diff-2c54e4ecf73ab55ca80a7497db6211001f27a916446694a6a988db65268216dbL97

* Formatting

* Fix change of name for init properties console.kt

* feat: ir ic (#604)

* Enable gzip

* process source module

* Support IC caches

* Init IC caches

* Prepare IC caches and copy it inside a request

* Fix main call arguments for JS IR with incremental compilation

* chore(release): add kotlin 1.8.20-RC

* Add js caches to buildLambda

* Revert "Add js caches to buildLambda"

This reverts commit e0c64d4.

* Revert "Fix main call arguments for JS IR with incremental compilation"

This reverts commit e8109a1.

* Revert "feat: ir ic (#604)"

This reverts commit dc4983e

* Use JS DCE to reduce output size

* chore(release): add kotlin 1.8.20

---------

Co-authored-by: Ilya Goncharov <[email protected]>
Co-authored-by: Mikhail Zarechenskiy <[email protected]>
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.

2 participants