mcpiroman
07/12/2022, 7:39 PMnatario1
07/13/2022, 1:46 PMchristophsturm
07/14/2022, 3:52 PMOleg Yukhnevich
07/15/2022, 11:26 AMpublic init
method, and supertype is from library where it internal init
- I would expect no error here (if not known, will create an issue)
2. New inference error [NewConstraintError at Incorporate TypeVariable(T) == ft<kotlin/Any, kotlin/Any?> from Fix variable TypeVariable(T) from position Fix variable TypeVariable(T): ft<kotlin/Any, kotlin/Any?> <!: CapturedType(*)?]
- have no idea what does it mean 🙂
3. val closeModalWindow = fun() { progressModalStage?.close() }
with K2 produces error: Return type mismatch: expected kotlin/Unit, actual kotlin/Unit? (not critical, as we can just specify type () -> Unit
explicitly for variable)
4. several issue with generics variances and java interop, but I have no idea how to create a small reproducer without internal codebase: Assignment type mismatch: actual type is ft<javafx/collections/ObservableList<out ft<OURTYPE, OURTYPE?>>, javafx/collections/ObservableList<out ft<OURTYPE, OURTYPE?>>?> but ft<javafx/collections/ObservableList<ft<CapturedType(out OURTYPE), CapturedType(out OURTYPE)?>>, javafx/collections/ObservableList<ft<CapturedType(out OURTYPE), CapturedType(out OURTYPE)?>>?> was expected
If something isn't known, I will create issues, but I will not be able to provide access to internal codebase.
FYI: project using java/javafx 17 and outdated tornadofx (which latest version was published with kotlin 1.3) - so issues can be from this weird stack alsoOleg Yukhnevich
07/15/2022, 11:27 AMHamza GATTAL
07/15/2022, 5:06 PMAkram Bensalem
07/16/2022, 8:32 AM-Xuse-k2
rkeazor
07/27/2022, 9:37 PMGrigory Panko
08/02/2022, 9:38 PMThere are some plugins incompatible with K2 compiler:
org.jetbrains.kotlin.lombok.LombokComponentRegistrar
Please remove -Xuse-k2
Note that this error is different from error in 1.7.10:
e: Compiler plugins are enabled with K2 compiler.
K2 does not support plugins yet, so please remove -Xuse-k2 flag
Did I forget to update anything?Ayfri
08/07/2022, 8:39 AMZac Sweers
08/07/2022, 9:02 PMZac Sweers
08/07/2022, 9:03 PMfinal
modifier in decompiled bytecode?
• If so, should plugin authors implement this functionality in their plugins? When would they choose to implement it in FIR vs IR in those cases?Gavin Ray
08/17/2022, 8:22 PMNew inference error [NewConstraintError at Incorporate TypeVariable(SELF) == CapturedType(*) from Fix variable TypeVariable(SELF) from position Fix variable TypeVariable(SELF): CapturedType(*) <!: ft<org/testcontainers/containers/MySQLContainer<ft<CapturedType(*) & Any, CapturedType(*)?>>, org/testcontainers/containers/MySQLContainer<ft<CapturedType(*) & Any, CapturedType(*)?>>?>]
Where that line is:
private val container = MySQLContainer(imageName).withInitScript(initScript)
And MySQL class definition is:
public class MySQLContainer<SELF extends MySQLContainer<SELF>> extends JdbcDatabaseContainer<SELF>
Gavin Ray
08/17/2022, 8:23 PMOleg Yukhnevich
08/23/2022, 8:19 PMrsocket-kotlin
to be jvm project instead of multiplatform: https://github.com/rsocket/rsocket-kotlin/tree/k2
And configured CI to run with and without K2 with different kotlin versions (CI run example here: https://github.com/rsocket/rsocket-kotlin/actions/runs/2914307633)
And found 2 issues already:
• https://youtrack.jetbrains.com/issue/KT-53698/K2-fails-to-resolve-suppressed-internal-symbol
• https://youtrack.jetbrains.com/issue/KT-53699/K2-Exception-during-IR-lowering-in-code-with-coroutines
Will continue creating issues if something new will happen
Also will try to setup similar setup for JS and Native, when K2 for them will be ready, until K2 multiplatform support
P.S. if K2 for JS/Native is already available with some highly experimental flag, please point on it, so I will setup such CI runs for them 🙂vineethraj49
08/24/2022, 4:36 AMDid K2 compilation went well?yep. few minor fixes for type inference (+-10 lines), and having to bump up to 1.7.20-Beta due to this bug but was fine otherwise.
Have you noticed a difference in the build speed?yep! our biggest module went from 44s to 20s on average so that’s great! overall, we see a ~25% improvement
Or what errors you encountered with, if they happened?1. aforementioned bug 2.
value?.property
on a definitely non-null value
was complaining, but am assuming it’s by design, so just fixed it
3. if both kotlin-stdlib-jdk8
and kotlin-stdlib-jre8
are somehow on path, the compiler complains about “confusing overloads”. excluding -jre8
worked.Marc Reichelt
08/24/2022, 1:03 PMZac Sweers
08/24/2022, 6:48 PMrrva
09/04/2022, 7:54 AMrrva
09/04/2022, 7:54 AMrrva
09/04/2022, 9:37 AMdmitriy.novozhilov
09/26/2022, 1:56 PMPHondogo
09/29/2022, 7:29 PMEric Ou
10/02/2022, 5:00 AMjava.lang.IllegalArgumentException: class org.jetbrains.kotlin.psi.KtParameter is not a subtype of class org.jetbrains.kotlin.psi.KtAnnotationEntry for factory REPEATED_ANNOTATION
Hey! I got this error when compiling with k2, which isn't thrown when I compile with the old compiler, lmk if you guys need more info to look into this!
P.S. This is the first time I've had problem with k2 since initial release, it is surprising stable.rrva
10/03/2022, 11:22 AMmitch
10/15/2022, 11:52 AMatsushieno
11/03/2022, 10:19 AMkotlinOptions
because they depend on Jetpack Compose (compiler plugin) but they still build (while they depend on other modules that are compiled with useK2 = true
). Any ideas why that happens?
Those builds reproduced as https://github.com/atsushieno/android-audio-plugin-framework/commit/2737fd77e176e866306329703b793e10bf8d76e7 (base) and https://github.com/atsushieno/aap-lv2/actions/runs/3384694415/jobs/5621953393#step:10:1347 (derived)Eric Ou
11/04/2022, 3:10 PMClass 'class' is compiled by a pre-release version of Kotlin and cannot be loaded by this version of the compiler
The this error is shown by the IDE despite the project using K2, and and project compiles and runs fine.Gavin Ray
11/10/2022, 6:03 PM"-Xuse-k2",
"-Xuse-fir-extended-checkers",
"-Xuse-fir-ic",
"-Xuse-fir-lt",
efemoney
11/18/2022, 2:34 PMefemoney
11/18/2022, 2:34 PM