From bbdaa8c3ff805d987b53b671214aa30f80df9396 Mon Sep 17 00:00:00 2001 From: Mikhail Glukhikh Date: Thu, 7 Sep 2023 11:08:53 +0200 Subject: [PATCH] LV 2.0: prevent double-set of LV in Native compilation tests --- .../kotlin/konan/blackboxtest/CompilerOutputTest.kt | 1 + .../blackboxtest/support/compilation/TestCompilation.kt | 9 +++++---- 2 files changed, 6 insertions(+), 4 deletions(-) diff --git a/native/native.tests/tests/org/jetbrains/kotlin/konan/blackboxtest/CompilerOutputTest.kt b/native/native.tests/tests/org/jetbrains/kotlin/konan/blackboxtest/CompilerOutputTest.kt index a8cada50fd5..a25e24261af 100644 --- a/native/native.tests/tests/org/jetbrains/kotlin/konan/blackboxtest/CompilerOutputTest.kt +++ b/native/native.tests/tests/org/jetbrains/kotlin/konan/blackboxtest/CompilerOutputTest.kt @@ -71,6 +71,7 @@ class CompilerOutputTest : AbstractNativeSimpleTest() { ): TestCompilationResult { val testCase = generateTestCaseWithSingleModule(source, TestCompilerArgs(freeCompilerArgs)) val compilation = LibraryCompilation( + pipelineType = null, settings = testRunSettings, freeCompilerArgs = testCase.freeCompilerArgs, sourceModules = testCase.modules, diff --git a/native/native.tests/tests/org/jetbrains/kotlin/konan/blackboxtest/support/compilation/TestCompilation.kt b/native/native.tests/tests/org/jetbrains/kotlin/konan/blackboxtest/support/compilation/TestCompilation.kt index be259142566..de7259edcc7 100644 --- a/native/native.tests/tests/org/jetbrains/kotlin/konan/blackboxtest/support/compilation/TestCompilation.kt +++ b/native/native.tests/tests/org/jetbrains/kotlin/konan/blackboxtest/support/compilation/TestCompilation.kt @@ -139,7 +139,7 @@ internal abstract class SourceBasedCompilation( private val gcType: GCType, private val gcScheduler: GCScheduler, private val allocator: Allocator, - private val pipelineType: PipelineType, + private val pipelineType: PipelineType?, freeCompilerArgs: TestCompilerArgs, compilerPlugins: CompilerPlugins, override val sourceModules: Collection, @@ -162,7 +162,7 @@ internal abstract class SourceBasedCompilation( sanitizer.compilerFlag?.let { compilerFlag -> add(compilerFlag) } gcType.compilerFlag?.let { compilerFlag -> add(compilerFlag) } gcScheduler.compilerFlag?.let { compilerFlag -> add(compilerFlag) } - pipelineType.compilerFlags.forEach { compilerFlag -> add(compilerFlag) } + pipelineType?.compilerFlags?.forEach { compilerFlag -> add(compilerFlag) } applyK2MPPArgs(this) } @@ -191,7 +191,8 @@ internal class LibraryCompilation( freeCompilerArgs: TestCompilerArgs, sourceModules: Collection, dependencies: Iterable>, - expectedArtifact: KLIB + expectedArtifact: KLIB, + pipelineType: PipelineType? = settings.get(), ) : SourceBasedCompilation( targets = settings.get(), home = settings.get(), @@ -203,7 +204,7 @@ internal class LibraryCompilation( gcType = settings.get(), gcScheduler = settings.get(), allocator = settings.get(), - pipelineType = settings.get(), + pipelineType = pipelineType, freeCompilerArgs = freeCompilerArgs, compilerPlugins = settings.get(), sourceModules = sourceModules,