From 813b600555fbbe7b03c02fcf14bea1849c68a92a Mon Sep 17 00:00:00 2001 From: Svyatoslav Kuzmich Date: Thu, 12 May 2022 17:52:00 +0300 Subject: [PATCH] [Wasm] Don't write .wat files in box tests These files take space on CI, but they are not validated or processed automatically in any way. Disabled by default, enabled via kotlin.wasm.debugMode=1 --- .../src/org/jetbrains/kotlin/backend/wasm/compiler.kt | 5 ++++- .../test/org/jetbrains/kotlin/js/testOld/BasicWasmBoxTest.kt | 2 +- 2 files changed, 5 insertions(+), 2 deletions(-) diff --git a/compiler/ir/backend.wasm/src/org/jetbrains/kotlin/backend/wasm/compiler.kt b/compiler/ir/backend.wasm/src/org/jetbrains/kotlin/backend/wasm/compiler.kt index a9956855a17..c6dc6121c5a 100644 --- a/compiler/ir/backend.wasm/src/org/jetbrains/kotlin/backend/wasm/compiler.kt +++ b/compiler/ir/backend.wasm/src/org/jetbrains/kotlin/backend/wasm/compiler.kt @@ -168,9 +168,12 @@ fun writeCompilationResult( dir: File, loaderKind: WasmLoaderKind, fileNameBase: String = "index", + writeWat: Boolean = true, ) { dir.mkdirs() - File(dir, "$fileNameBase.wat").writeText(result.wat) + if (writeWat) { + File(dir, "$fileNameBase.wat").writeText(result.wat) + } File(dir, "$fileNameBase.wasm").writeBytes(result.wasm) val jsWithLoader = generateJsWasmLoader(loaderKind, "./$fileNameBase.wasm", result.js) File(dir, "$fileNameBase.js").writeText(jsWithLoader) diff --git a/js/js.tests/test/org/jetbrains/kotlin/js/testOld/BasicWasmBoxTest.kt b/js/js.tests/test/org/jetbrains/kotlin/js/testOld/BasicWasmBoxTest.kt index a396eddbfac..d5ccde54ed0 100644 --- a/js/js.tests/test/org/jetbrains/kotlin/js/testOld/BasicWasmBoxTest.kt +++ b/js/js.tests/test/org/jetbrains/kotlin/js/testOld/BasicWasmBoxTest.kt @@ -176,7 +176,7 @@ abstract class BasicWasmBoxTest( println(" ------ $name Test file://$path/test.js") } - writeCompilationResult(res, dir, WasmLoaderKind.D8) + writeCompilationResult(res, dir, WasmLoaderKind.D8, writeWat = debugMode >= DebugMode.DEBUG) File(dir, "test.js").writeText(testJs) ExternalTool(System.getProperty("javascript.engine.path.V8")) .run(