Skip to content

Commit fda2fdd

Browse files
committed
Where's my .asm.txt and .fir.ir.txt files?
1 parent caa480f commit fda2fdd

File tree

14 files changed

+70
-1166
lines changed

14 files changed

+70
-1166
lines changed

README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1068,7 +1068,7 @@ copies or substantial portions of the Software.
10681068
10691069
THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
10701070
IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
1071-
FITNESS FOR love.forte.plugin.suspendtrans.A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
1071+
FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
10721072
AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
10731073
LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
10741074
OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE

README_CN.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -752,7 +752,7 @@ copies or substantial portions of the Software.
752752
753753
THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
754754
IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
755-
FITNESS FOR love.forte.plugin.suspendtrans.A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
755+
FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
756756
AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
757757
LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
758758
OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE

compiler/suspend-transform-plugin-cli/src/main/kotlin/love/forte/plugin/suspendtrans/cli/SuspendTransformCliOption.kt

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -35,8 +35,8 @@ private data class AbstractCliOptionImpl(
3535
* Creates an instance of [SuspendTransformCliOption] to describe and define a CLI option.
3636
*
3737
* @param optionName The name of the option used to identify it in the CLI.
38-
* @param valueDescription love.forte.plugin.suspendtrans.A description of the option's value, defaults to the option name.
39-
* @param description love.forte.plugin.suspendtrans.A textual description of the option, defaults to an empty string.
38+
* @param valueDescription A description of the option's value, defaults to the option name.
39+
* @param description A textual description of the option, defaults to an empty string.
4040
* @param required Whether this option is mandatory, defaults to not required (`false`).
4141
* @param allowMultipleOccurrences Whether this option can appear multiple times in the CLI,
4242
* defaults to not allowed (`false`).

compiler/suspend-transform-plugin/build.gradle.kts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,7 @@ dependencies {
1616
api(project(":compiler:suspend-transform-plugin-deprecated-configuration"))
1717
api(project(":compiler:suspend-transform-plugin-cli"))
1818

19+
testImplementation("junit:junit:4.13.2")
1920
testImplementation(kotlin("stdlib"))
2021
testImplementation(kotlin("test-junit5"))
2122

compiler/suspend-transform-plugin/src/main/kotlin/love/forte/plugin/suspendtrans/fir/SuspendTransformFirTransformer.kt

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -385,7 +385,7 @@ class SuspendTransformFirTransformer(
385385
source = thisReceiverParameter.source
386386
calleeReference = buildImplicitThisReference {
387387
boundSymbol = thisReceiverParameter.symbol
388-
println("[${newFunSymbol}] thisReceiverParameter.symbol: ${thisReceiverParameter.symbol}")
388+
// println("[${newFunSymbol}] thisReceiverParameter.symbol: ${thisReceiverParameter.symbol}")
389389
}
390390
}
391391
}
@@ -608,11 +608,11 @@ class SuspendTransformFirTransformer(
608608

609609
// Copy the typeParameters.
610610
// Otherwise, in functions like the following, an error will occur
611-
// suspend fun <love.forte.plugin.suspendtrans.A> data(value: love.forte.plugin.suspendtrans.A): T = ...
612-
// Functions for which function-scoped generalizations (`<love.forte.plugin.suspendtrans.A>`) exist.
613-
// In the generated IR, data and dataBlocking will share an `love.forte.plugin.suspendtrans.A`, generating the error.
611+
// suspend fun <A> data(value: A): T = ...
612+
// Functions for which function-scoped generalizations (`<A>`) exist.
613+
// In the generated IR, data and dataBlocking will share an `A`, generating the error.
614614
// The error: Duplicate IR node
615-
// [IR VALIDATION] JvmIrValidationBeforeLoweringPhase: Duplicate IR node: TYPE_PARAMETER name:love.forte.plugin.suspendtrans.A index:0 variance: superTypes:[kotlin.Any?] reified:false of FUN GENERATED[...]
615+
// [IR VALIDATION] JvmIrValidationBeforeLoweringPhase: Duplicate IR node: TYPE_PARAMETER name:A index:0 variance: superTypes:[kotlin.Any?] reified:false of FUN GENERATED[...]
616616
copyParameters()
617617

618618
// resolve returnType (with wrapped) after copyParameters

compiler/suspend-transform-plugin/src/test/love/forte/plugin/suspendtrans/runners/AbstractCodeGenTestRunner.kt

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,10 @@ abstract class AbstractCodeGenTestRunner : AbstractTestRunner() {
1717
override fun TestConfigurationBuilder.configureHandlers() {
1818
configureFirHandlersStep {
1919
useHandlers(
20-
::FirDumpHandler, ::FirCfgDumpHandler, ::FirResolvedTypesVerifier, ::FirDiagnosticsHandler,
20+
::FirDumpHandler,
21+
::FirCfgDumpHandler,
22+
::FirResolvedTypesVerifier,
23+
::FirDiagnosticsHandler,
2124
)
2225
}
2326

compiler/suspend-transform-plugin/src/test/love/forte/plugin/suspendtrans/runners/AbstractTestRunner.kt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -42,6 +42,7 @@ abstract class AbstractTestRunner : AbstractKotlinCompilerTest() {
4242
backendKind = BackendKinds.IrBackendForK1AndK2
4343
// this.languageVersionSettings
4444

45+
4546
// languageSettings {
4647
// languageVersion = LanguageVersion.KOTLIN_2_1
4748
// apiVersion = ApiVersion.KOTLIN_2_1
@@ -54,7 +55,6 @@ abstract class AbstractTestRunner : AbstractKotlinCompilerTest() {
5455

5556
builder.defaultDirectives {
5657
FIR_PARSER with FirParser.LightTree
57-
5858
}
5959

6060
when (targetFrontend) {

0 commit comments

Comments
 (0)