BridgeJS: Snapshot for LIFO order issue in Swift glue code for stack-based types

Author: krodak

Plugins/BridgeJS/Tests/BridgeJSToolTests/Inputs/EnumAssociatedValue.swift

@@ -10,12 +10,8 @@ enum APIResult {
 
 @JS func handle(result: APIResult)
 @JS func getResult() -> APIResult
-@JS func roundtripAPIResult(result: APIResult) -> APIResult {
-    return result
-}
-@JS func roundTripOptionalAPIResult(result: APIResult?) -> APIResult? {
-    return result
-}
+@JS func roundtripAPIResult(result: APIResult) -> APIResult
+@JS func roundTripOptionalAPIResult(result: APIResult?) -> APIResult?
 
 @JS
 enum ComplexResult {
@@ -29,12 +25,8 @@ enum ComplexResult {
 
 @JS func handleComplex(result: ComplexResult)
 @JS func getComplexResult() -> ComplexResult
-@JS func roundtripComplexResult(_ result: ComplexResult) -> ComplexResult {
-    return result
-}
-@JS func roundTripOptionalComplexResult(result: ComplexResult?) -> ComplexResult? {
-    return result
-}
+@JS func roundtripComplexResult(_ result: ComplexResult) -> ComplexResult
+@JS func roundTripOptionalComplexResult(result: ComplexResult?) -> ComplexResult?
 
 @JS
 enum Utilities {
@@ -45,26 +37,21 @@ enum Utilities {
     }
 }
 
-@JS func roundTripOptionalUtilitiesResult(result: Utilities.Result?) -> Utilities.Result? {
-    return result
-}
+@JS func roundTripOptionalUtilitiesResult(result: Utilities.Result?) -> Utilities.Result?
 
 @JS(namespace: "API")
 @JS enum NetworkingResult {
     case success(String)
     case failure(String, Int)
 }
 
-@JS func roundTripOptionalNetworkingResult(result: NetworkingResult?) -> NetworkingResult? {
-    return result
-}
+@JS func roundTripOptionalNetworkingResult(result: NetworkingResult?) -> NetworkingResult? 
 
 @JS
 enum APIOptionalResult {
     case success(String?)
     case failure(Int?, Bool?)
     case status(Bool?, Int?, String?)
 }
-@JS func roundTripOptionalAPIOptionalResult(result: APIOptionalResult?) -> APIOptionalResult? {
-    return result
-}
+@JS func roundTripOptionalAPIOptionalResult(result: APIOptionalResult?) -> APIOptionalResult?
+@JS func compareAPIResults(result1: APIOptionalResult?, result2: APIOptionalResult?) -> APIOptionalResult?

Plugins/BridgeJS/Tests/BridgeJSToolTests/__Snapshots__/BridgeJSLinkTests/EnumAssociatedValue.Export.d.ts

@@ -86,6 +86,7 @@ export type Exports = {
     roundTripOptionalUtilitiesResult(result: Utilities.ResultTag | null): Utilities.ResultTag | null;
     roundTripOptionalNetworkingResult(result: API.NetworkingResultTag | null): API.NetworkingResultTag | null;
     roundTripOptionalAPIOptionalResult(result: APIOptionalResultTag | null): APIOptionalResultTag | null;
+    compareAPIResults(result1: APIOptionalResultTag | null, result2: APIOptionalResultTag | null): APIOptionalResultTag | null;
     APIResult: APIResultObject
     ComplexResult: ComplexResultObject
     APIOptionalResult: APIOptionalResultObject

Plugins/BridgeJS/Tests/BridgeJSToolTests/__Snapshots__/BridgeJSLinkTests/EnumAssociatedValue.Export.js

@@ -814,6 +814,33 @@ export async function createInstantiator(options, swift) {
                     if (resultCleanup) { resultCleanup(); }
                     return optResult;
                 },
+                compareAPIResults: function bjs_compareAPIResults(result1, result2) {
+                    const isSome = result1 != null;
+                    let result1CaseId, result1Cleanup;
+                    if (isSome) {
+                        const enumResult = enumHelpers.APIOptionalResult.lower(result1);
+                        result1CaseId = enumResult.caseId;
+                        result1Cleanup = enumResult.cleanup;
+                    }
+                    const isSome1 = result2 != null;
+                    let result2CaseId, result2Cleanup;
+                    if (isSome1) {
+                        const enumResult1 = enumHelpers.APIOptionalResult.lower(result2);
+                        result2CaseId = enumResult1.caseId;
+                        result2Cleanup = enumResult1.cleanup;
+                    }
+                    instance.exports.bjs_compareAPIResults(+isSome, isSome ? result1CaseId : 0, +isSome1, isSome1 ? result2CaseId : 0);
+                    const isNull = (tmpRetTag === -1);
+                    let optResult;
+                    if (isNull) {
+                        optResult = null;
+                    } else {
+                        optResult = enumHelpers.APIOptionalResult.raise(tmpRetTag, tmpRetStrings, tmpRetInts, tmpRetF32s, tmpRetF64s);
+                    }
+                    if (result1Cleanup) { result1Cleanup(); }
+                    if (result2Cleanup) { result2Cleanup(); }
+                    return optResult;
+                },
                 APIResult: APIResultValues,
                 ComplexResult: ComplexResultValues,
                 APIOptionalResult: APIOptionalResultValues,

Plugins/BridgeJS/Tests/BridgeJSToolTests/__Snapshots__/ExportSwiftTests/EnumAssociatedValue.json

@@ -787,6 +787,52 @@
           }
         }
       }
+    },
+    {
+      "abiName" : "bjs_compareAPIResults",
+      "effects" : {
+        "isAsync" : false,
+        "isStatic" : false,
+        "isThrows" : false
+      },
+      "name" : "compareAPIResults",
+      "parameters" : [
+        {
+          "label" : "result1",
+          "name" : "result1",
+          "type" : {
+            "optional" : {
+              "_0" : {
+                "associatedValueEnum" : {
+                  "_0" : "APIOptionalResult"
+                }
+              }
+            }
+          }
+        },
+        {
+          "label" : "result2",
+          "name" : "result2",
+          "type" : {
+            "optional" : {
+              "_0" : {
+                "associatedValueEnum" : {
+                  "_0" : "APIOptionalResult"
+                }
+              }
+            }
+          }
+        }
+      ],
+      "returnType" : {
+        "optional" : {
+          "_0" : {
+            "associatedValueEnum" : {
+              "_0" : "APIOptionalResult"
+            }
+          }
+        }
+      }
     }
   ],
   "moduleName" : "TestModule",

Plugins/BridgeJS/Tests/BridgeJSToolTests/__Snapshots__/ExportSwiftTests/EnumAssociatedValue.swift

@@ -606,4 +606,15 @@ public func _bjs_roundTripOptionalAPIOptionalResult(resultIsSome: Int32, resultC
     #else
     fatalError("Only available on WebAssembly")
     #endif
+}
+
+@_expose(wasm, "bjs_compareAPIResults")
+@_cdecl("bjs_compareAPIResults")
+public func _bjs_compareAPIResults(result1IsSome: Int32, result1CaseId: Int32, result2IsSome: Int32, result2CaseId: Int32) -> Void {
+    #if arch(wasm32)
+    let ret = compareAPIResults(result1: Optional<APIOptionalResult>.bridgeJSLiftParameter(result1IsSome, result1CaseId), result2: Optional<APIOptionalResult>.bridgeJSLiftParameter(result2IsSome, result2CaseId))
+    return ret.bridgeJSLowerReturn()
+    #else
+    fatalError("Only available on WebAssembly")
+    #endif
 }