[cDAC] Ensure cDAC is used in test runs

src/coreclr/debug/daccess/daccess.cpp

@@ -7045,6 +7045,12 @@ CLRDataCreateInstance(REFIID iid,
                 // Release the AddRef from the QI.
                 pClrDataAccess->Release();
             }
+
+            if (cdacInterface == nullptr)
+            {
+                // If we requested to use the cDAC, but failed to create the cDAC interface, return failure
+                return E_FAIL;
+            }
         }
     }
 #endif

src/installer/pkg/sfx/Microsoft.NETCore.App/Directory.Build.props

@@ -115,6 +115,9 @@
     <PlatformManifestFileEntry Condition="'$(PgoInstrument)' != ''" Include="clrjit.pgd" IsNative="true" />
     <PlatformManifestFileEntry Include="libclrjit.so" IsNative="true" />
     <PlatformManifestFileEntry Include="libclrjit.dylib" IsNative="true" />
+    <PlatformManifestFileEntry Include="mscordaccore_universal.dll" IsNative="true" />
+    <PlatformManifestFileEntry Include="libmscordaccore_universal.so" IsNative="true" />
+    <PlatformManifestFileEntry Include="libmscordaccore_universal.dylib" IsNative="true" />
     <PlatformManifestFileEntry Include="mscordaccore.dll" IsNative="true" />
     <PlatformManifestFileEntry Include="libmscordaccore.so" IsNative="true" />
     <PlatformManifestFileEntry Include="libmscordaccore.dylib" IsNative="true" />

src/native/managed/cdac/Microsoft.Diagnostics.DataContractReader.Contracts/Contracts/StackWalk/Context/AMD64/AMD64Unwinder.cs

@@ -303,15 +303,17 @@ public bool Unwind(ref AMD64Context context)
                 branchTarget = nextByte - imageBase;
                 if (ReadByteAt(nextByte) == JMP_IMM8_OP)
                 {
-                    branchTarget += 2u + ReadByteAt(nextByte + 1);
+                    // sign-extend the 8-bit immediate value
+                    branchTarget += 2u + (ulong)(sbyte)ReadByteAt(nextByte + 1);
                 }
                 else
                 {
+                    // sign-extend the 32-bit immediate value
                     int delta = ReadByteAt(nextByte + 1) |
                                 (ReadByteAt(nextByte + 2) << 8) |
                                 (ReadByteAt(nextByte + 3) << 16) |
                                 (ReadByteAt(nextByte + 4) << 24);
-                    branchTarget += (uint)(5 + delta);
+                    branchTarget += (ulong)(5 + delta);
                 }
 
                 //

src/native/managed/cdac/Microsoft.Diagnostics.DataContractReader.Contracts/Contracts/StackWalk/Context/AMD64Context.cs

@@ -31,7 +31,7 @@ public enum ContextFlagsValues : uint
     }
 
     public readonly uint Size => 0x4d0;
-    public readonly uint DefaultContextFlags => (uint)ContextFlagsValues.CONTEXT_FULL;
+    public readonly uint DefaultContextFlags => (uint)ContextFlagsValues.CONTEXT_ALL;
 
     public TargetPointer StackPointer
     {

src/native/managed/cdac/Microsoft.Diagnostics.DataContractReader.Contracts/Contracts/StackWalk/Context/ARM64Context.cs

@@ -38,7 +38,10 @@ public enum ContextFlagsValues : uint
 
     public readonly uint Size => 0x390;
 
-    public readonly uint DefaultContextFlags => (uint)ContextFlagsValues.CONTEXT_FULL;
+    public readonly uint DefaultContextFlags => (uint)(ContextFlagsValues.CONTEXT_CONTROL |
+                                                       ContextFlagsValues.CONTEXT_INTEGER |
+                                                       ContextFlagsValues.CONTEXT_FLOATING_POINT |
+                                                       ContextFlagsValues.CONTEXT_DEBUG_REGISTERS);
 
     public TargetPointer StackPointer
     {

src/native/managed/cdac/Microsoft.Diagnostics.DataContractReader.Contracts/Contracts/StackWalk/Context/ARMContext.cs

@@ -29,7 +29,7 @@ public enum ContextFlagsValues : uint
     }
 
     public readonly uint Size => 0x1a0;
-    public readonly uint DefaultContextFlags => (uint)ContextFlagsValues.CONTEXT_FULL;
+    public readonly uint DefaultContextFlags => (uint)ContextFlagsValues.CONTEXT_ALL;
 
     public TargetPointer StackPointer
     {

src/native/managed/cdac/Microsoft.Diagnostics.DataContractReader.Contracts/Contracts/StackWalk/Context/X86Context.cs

@@ -22,8 +22,9 @@ public enum ContextFlagsValues : uint
         CONTEXT_SEGMENTS = CONTEXT_i386 | 0x4,
         CONTEXT_FLOATING_POINT = CONTEXT_i386 | 0x8,
         CONTEXT_DEBUG_REGISTERS = CONTEXT_i386 | 0x10,
+        CONTEXT_EXTENDED_REGISTERS = CONTEXT_i386 | 0x20,
         CONTEXT_FULL = CONTEXT_CONTROL | CONTEXT_INTEGER | CONTEXT_FLOATING_POINT,
-        CONTEXT_ALL = CONTEXT_CONTROL | CONTEXT_INTEGER | CONTEXT_SEGMENTS | CONTEXT_FLOATING_POINT | CONTEXT_DEBUG_REGISTERS,
+        CONTEXT_ALL = CONTEXT_CONTROL | CONTEXT_INTEGER | CONTEXT_SEGMENTS | CONTEXT_FLOATING_POINT | CONTEXT_DEBUG_REGISTERS | CONTEXT_EXTENDED_REGISTERS,
         CONTEXT_XSTATE = CONTEXT_i386 | 0x40,
 
         //
@@ -37,7 +38,7 @@ public enum ContextFlagsValues : uint
     }
 
     public readonly uint Size => 0x2cc;
-    public readonly uint DefaultContextFlags => (uint)ContextFlagsValues.CONTEXT_FULL;
+    public readonly uint DefaultContextFlags => (uint)ContextFlagsValues.CONTEXT_ALL;
 
     public TargetPointer StackPointer
     {

src/native/managed/cdac/Microsoft.Diagnostics.DataContractReader.Contracts/Contracts/StackWalk/StackWalk_1.cs

@@ -191,6 +191,11 @@ private unsafe void FillContextFromThread(IPlatformAgnosticContext context, Thre
     {
         byte[] bytes = new byte[context.Size];
         Span<byte> buffer = new Span<byte>(bytes);
+        // The underlying ICLRDataTarget.GetThreadContext has some variance depending on the host.
+        // SOS's managed implementation sets the ContextFlags to platform specific values defined in ThreadService.cs (diagnostics repo)
+        // SOS's native implementation keeps the ContextFlags passed into this function.
+        // To match the DAC behavior, the DefaultContextFlags are what the DAC passes in in DacGetThreadContext.
+        // In most implementations, this will be overridden by the host, but in some cases, it may not be.
         if (!_target.TryGetThreadContext(threadData.OSId.Value, context.DefaultContextFlags, buffer))
         {
             throw new InvalidOperationException($"GetThreadContext failed for thread {threadData.OSId.Value}");

src/native/managed/cdac/mscordaccore_universal/mscordaccore_universal.csproj

@@ -27,8 +27,7 @@
   </ItemGroup>
   <ItemGroup>
     <InstallRuntimeComponentDestination Include="." />
-    <!-- TODO: [cdac] Output to sharedFramework and add PlatformManifestFileEntry for Microsoft.NETCore.App once ready to include in shipping package -->
-    <!-- <InstallRuntimeComponentDestination Include="sharedFramework" Condition="'$(RuntimeFlavor)' == 'coreclr'"/> -->
+    <InstallRuntimeComponentDestination Include="sharedFramework" Condition="'$(RuntimeFlavor)' == 'coreclr'"/>
   </ItemGroup>
 
   <ItemGroup>