Skip to content

Commit

Permalink
Made nativeIterationMode advanced option
Browse files Browse the repository at this point in the history
  • Loading branch information
Elena Lepilkina authored and Elena Lepilkina committed Aug 5, 2021
1 parent e2b2d91 commit 0da9663
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 3 deletions.
2 changes: 1 addition & 1 deletion examples/kotlin-multiplatform/build.gradle
Expand Up @@ -98,7 +98,7 @@ benchmark {
iterationTime = 300 // time in ms per iteration
iterationTimeUnit = "ms" // time in ms per iteration
advanced("forks", 1)
nativeIterationMode = "internal"
advanced("nativeIterationMode", "internal")
nativeGCCollectMode = "iteration"
}

Expand Down
Expand Up @@ -9,7 +9,6 @@ open class BenchmarkConfiguration(val extension: BenchmarksExtension, val name:
var iterationTime: Long? = null
var iterationTimeUnit: String? = null
var mode: String? = null
var nativeIterationMode: String? = null // TODO: where should warning about K/N specific of this parameter be shown?
var nativeGCCollectMode: String? = null
var outputTimeUnit: String? = null
var reportFormat: String? = null
Expand All @@ -36,6 +35,9 @@ open class BenchmarkConfiguration(val extension: BenchmarksExtension, val name:
advanced[name] = value
}

val nativeIterationMode: String?
get() = advanced["nativeIterationMode"] as? String

fun capitalizedName() = if (name == "main") "" else name.capitalize()
fun prefixName(suffix: String) = if (name == "main") suffix else name + suffix.capitalize()
fun reportFileExt(): String = reportFormat?.toLowerCase() ?: "json"
Expand Down
1 change: 0 additions & 1 deletion plugin/main/src/kotlinx/benchmark/gradle/Utils.kt
Expand Up @@ -94,7 +94,6 @@ fun writeParameters(
config.iterationTimeUnit?.let { appendln("iterationTimeUnit:$it") }
config.outputTimeUnit?.let { appendln("outputTimeUnit:$it") }
config.mode?.let { appendln("mode:$it") }
config.nativeIterationMode?.let { appendln("nativeIterationMode:$it") }
config.nativeGCCollectMode?.let { appendln("nativeGCCollectMode:$it") }

config.includes.forEach {
Expand Down

0 comments on commit 0da9663

Please sign in to comment.