Plugin: Remove Kotlin Reflect dependency

[adam-enko]

From what I can tell, the Kotlin Reflect dependency is unnecessary.

To avoid clashes with Gradle's embedded Kotlin, and with other plugins, Gradle plugins should avoid using unnecessary dependencies.

[fzhinkin]

There's in fact a single explicit usage of the reflect library in the plugin. :)

In general, it seems like the proper should be similar to what was done in #235 for compiler-embeddable, and dependencies like kotlinpoet, reflect, and klib-related stuff should all be moved into a separate configuration.

[adam-enko]

There's in fact a single explicit usage of the reflect library in the plugin. :)

Please could you share it here?

[fzhinkin]

kotlinx-benchmark/plugin/main/src/kotlinx/benchmark/gradle/JmhBytecodeGeneratorWorker.kt

Line 187 in e9becb5

"${holder.kotlin.qualifiedName ?: holder.name}.${function.kotlinFunction?.name ?: function.name}"

[adam-enko]

kotlinx-benchmark/plugin/main/src/kotlinx/benchmark/gradle/JmhBytecodeGeneratorWorker.kt

Line 187 in e9becb5

"${holder.kotlin.qualifiedName ?: holder.name}.${function.kotlinFunction?.name ?: function.name}"

Thanks! This code is inside a Worker, so the classpath should be isolated. This means the dependency can be changed from implementation to compileOnly.

kotlinx-benchmark/plugin/build.gradle.kts

Line 108 in e9becb5

implementation(libs.kotlin.reflect)