-
Notifications
You must be signed in to change notification settings - Fork 5.5k
Comparing changes
Open a pull request
base repository: dotnet/runtime
base: 0a5cccb579
head repository: dotnet/runtime
compare: d0f3235d31
- 6 commits
- 23 files changed
- 9 contributors
Commits on Aug 29, 2024
-
[release/9.0] Ensure the layout of
StackAllocatedBox<T>matches box……ed layouts (#107075) * Ensure the layout of `StackAllocatedBox<T>` matches boxed layouts The boxed layout of a struct always has its data at +8, as evidenced by `Object::UnBox`. This means that `StackAllocatedBox<T>` should have `Pack = 1`, otherwise this may not be the case. In the test failure we had a `StackAllocatedBox<Int128>` which had its `_value` field at offset 16. After object stack allocation this meant that we were saving data in padding of the structure, which promotion does not guarantee to preserve. Fix #106947 * Run jit-format * Correct enum --------- Co-authored-by: Jakob Botsch Nielsen <jakob.botsch.nielsen@gmail.com> Co-authored-by: Jeff Schwartz <jeffschw@microsoft.com>
Configuration menu - View commit details
-
Copy full SHA for 6c8b1ed - Browse repository at this point
Copy the full SHA 6c8b1edView commit details -
Ensure that AdvSimd.Insert doesn't zero out the upper bits (#107089)
Co-authored-by: Tanner Gooding <tagoo@outlook.com> Co-authored-by: Jeff Schwartz <jeffschw@microsoft.com>
Configuration menu - View commit details
-
Copy full SHA for c827c27 - Browse repository at this point
Copy the full SHA c827c27View commit details -
Fix storage of stack trace of exception from reflection (#107093)
There was one more case where we have saved the stack trace into the _remoteStackTraceString field in the exception when the exception was passing from reflection invoked code to the caller of that code. While there is no visible difference in the Exception.ToString, a SOS test was failing due to that. And it is not necessary to save the stack trace there. I have thought about the cases when we really need the stack trace saved into the _remoteStackTraceString and I believe that actually the only case is when an existing exception is thrown again in managed code. So I have removed the option to save the stack trace from all the variants of the DispatchManagedException. Co-authored-by: Jan Vorlicek <janvorli@microsoft.com> Co-authored-by: Jeff Schwartz <jeffschw@microsoft.com>
Configuration menu - View commit details
-
Copy full SHA for 13c6579 - Browse repository at this point
Copy the full SHA 13c6579View commit details -
[release/9.0] Tensor Operation fixes when input is sliced (non-contig…
…uous) (#107097) * slicing fixes * rebased on main --------- Co-authored-by: Michael Sharp <misharp@microsoft.com>
Configuration menu - View commit details
-
Copy full SHA for 00f3f3e - Browse repository at this point
Copy the full SHA 00f3f3eView commit details -
[release/9.0] Azure Linux 3.0 and test fixes (#107123)
* Change BuildWithFactoryReadDirect to not use RSA+MD5 * Add Azure 3.0 helix test --------- Co-authored-by: Kevin Jones <kevin@vcsjones.com> Co-authored-by: Rich Lander <rlander@microsoft.com>
Configuration menu - View commit details
-
Copy full SHA for 33a17bb - Browse repository at this point
Copy the full SHA 33a17bbView commit details
Commits on Aug 30, 2024
-
Use the typical MethodTable not the instantiated one. (#107160)
Co-authored-by: Aaron R Robinson <arobins@microsoft.com>
Configuration menu - View commit details
-
Copy full SHA for d0f3235 - Browse repository at this point
Copy the full SHA d0f3235View commit details
This comparison is taking too long to generate.
Unfortunately it looks like we can’t render this comparison for you right now. It might be too big, or there might be something weird with your repository.
You can try running this command locally to see the comparison on your machine:
git diff 0a5cccb579...d0f3235d31