Move the ZipFIleSystemAccessor configuration from JSConfiguration to common klib configurations and add compiler arguments.
Enabling cached ZipFileSystemAccessor for Kotlin Native.
Closes: https://youtrack.jetbrains.com/issue/KT-79675/
There are two "merge" commits which are undesirable. The single meaningful commit should be added as a result of rebasing on top of fresh master branch.
I would highly recommend to split the commit into two separate commits:
- The one that makes ZIP FS accessor available not only for JS but also for all KLIB-based compilers.
- The one that adds fine-tuning.
The commit message should contain a YT ticket number. It's better to put it onto a separate line. Example:
Some explanation.
^KT-00000
I think, it worth adding KlibLoaderSpec.zipFileSystemAccessor() calls to:
-
loadMetadataKlibs()- https://github.com/JetBrains/kotlin/blob/6d55c67eef914cb1ee8b9fa5cd4369446723f962/compiler/ir/serialization.common/src/org/jetbrains/kotlin/backend/common/loadMetadataKlibs.kt#L17 -
loadNativeKlibsInTestPipeline()- https://github.com/JetBrains/kotlin/blob/6d55c67eef914cb1ee8b9fa5cd4369446723f962/compiler/ir/serialization.native/src/org/jetbrains/kotlin/backend/konan/serialization/loadNativeKlibs.kt#L30
GH shows that there is a source conflict. Please rebase on fresh master, then I will run IC internally.