Revert "Fix for missing trace event on first expression (#281)"

This reverts commit ce7ce74.
IronLanguages · Nov 27, 2024 · 1840445 · 1840445
1 parent ce7ce74
commit 1840445
Show file tree

Hide file tree

Showing 3 changed files with 20 additions and 15 deletions.
diff --git a/.github/workflows/main.yml b/.github/workflows/main.yml
@@ -13,31 +13,31 @@ jobs:
         os: [windows-latest, ubuntu-latest, macos-latest-large]
 
     steps:
-    - uses: actions/checkout@v4
+    - uses: actions/checkout@v2
     - name: Setup .NET Core 3.1
-      uses: actions/setup-dotnet@v4
+      uses: actions/setup-dotnet@v1
       with:
         dotnet-version: '3.1.x'
     - name: Setup .NET 6.0
-      uses: actions/setup-dotnet@v4
+      uses: actions/setup-dotnet@v1
       with:
         dotnet-version: '6.0.x'
     - name: Setup .NET 8.0
-      uses: actions/setup-dotnet@v4
+      uses: actions/setup-dotnet@v1
       with:
         dotnet-version: '8.0.x'
     - name: Setup .NET 9.0
-      uses: actions/setup-dotnet@v4
+      uses: actions/setup-dotnet@v1
       with:
         dotnet-version: '9.0.x'
         include-prerelease: true
     - name: Build
       run: pwsh make.ps1
     - name: Package
       run: pwsh make.ps1 package
-    - uses: actions/upload-artifact@v4
+    - uses: actions/upload-artifact@v2
       with:
-        name: packages-${{ matrix.os }}
+        name: packages
         path: Package/Release/Packages
     - name: Test (net462)
       run: ./make.ps1 -frameworks net462 test-all

diff --git a/Directory.Build.props b/Directory.Build.props
@@ -94,7 +94,7 @@
 
   <!-- Release -->
   <PropertyGroup Condition="'$(Configuration)' == 'Release'">
-    <DebugSymbols>true</DebugSymbols>
+    <DebugSymbols>false</DebugSymbols>
     <DebugType>portable</DebugType>
     <Optimize>true</Optimize>
     <CheckForOverflowUnderflow>false</CheckForOverflowUnderflow>

diff --git a/Src/Microsoft.Dynamic/Debugging/DebugInfoRewriter.cs b/Src/Microsoft.Dynamic/Debugging/DebugInfoRewriter.cs
@@ -17,7 +17,7 @@
 namespace Microsoft.Scripting.Debugging {
     using Ast = MSAst.Expression;
     using System.Threading;
-
+    
     /// <summary>
     /// Used to rewrite expressions containing DebugInfoExpressions.
     /// </summary>
@@ -355,12 +355,17 @@ protected override MSAst.Expression VisitDebugInfo(MSAst.DebugInfoExpression nod
                 // Update the location cookie
                 int locationCookie = _locationCookie++;
                 if (!_transformToGenerator) {
-                    var tracebackCall = Ast.Call(
-                        typeof(RuntimeOps).GetMethod(nameof(RuntimeOps.OnTraceEvent)),
-                        _thread,
-                        AstUtils.Constant(locationCookie),
-                        Ast.Convert(Ast.Constant(null), typeof(Exception))
-                    );
+                    MSAst.Expression tracebackCall = null;
+                    if (locationCookie == 0) {
+                        tracebackCall = Ast.Empty();
+                    } else {
+                        tracebackCall = Ast.Call(
+                            typeof(RuntimeOps).GetMethod(nameof(RuntimeOps.OnTraceEvent)),
+                            _thread,
+                            AstUtils.Constant(locationCookie),
+                            Ast.Convert(Ast.Constant(null), typeof(Exception))
+                        );
+                    }
 
                     transformedExpression = Ast.Block(
                         Ast.Assign(