Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

JIT: Fix assertion due to remorph #110516

Closed
wants to merge 70 commits into from
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
70 commits
Select commit Hold shift + click to select a range
0cbf3a2
Try fixing remorph issue
hez2010 Dec 6, 2024
8e63d8d
Delete unused static field from List (#110515)
stephentoub Dec 9, 2024
f7fad34
Remove some RuntimeExport/RuntimeImport indirections (#110437)
MichalStrehovsky Dec 9, 2024
9df306f
Fix linux-armel build (#110514)
am11 Dec 9, 2024
d9f5ee4
[wasi] bump wasmtime to 27 (#110524)
pavelsavara Dec 9, 2024
4525619
[debugger] Fix a step that becomes a go (#110484)
thaystg Dec 9, 2024
df70878
fix profiling env var names in profiling.md (#109764)
loadingcn Dec 9, 2024
6ade56a
Change assertion in IPGlobalProperties_DomainName_ReturnsEmptyStringW…
antonfirsov Dec 9, 2024
1c960dd
Added a fix for the build failure when STRESS_DYNAMIC_HEAP_COUNT is d…
mrsharm Dec 9, 2024
6fdfb78
[cdac] Handle no method def token when trying to get the IL version s…
elinor-fung Dec 9, 2024
a08711c
Normalization APIs using the spans (#110465)
tarekgh Dec 9, 2024
511b2e0
Fix test under JIT stress (#110538)
AaronRobinsonMSFT Dec 10, 2024
810bad5
JIT: extract BBJ_COND to BBJ_ALWAYS profile repair as utility (#110494)
AndyAyersMS Dec 10, 2024
cf92a64
Speed up surrogate validation in HttpUtility (#110478)
MihaZupan Dec 10, 2024
69229cd
Don't wait for finalizers in 'IReferenceTrackerHost::ReleaseDisconnec…
Sergio0694 Dec 10, 2024
dede4f9
[cdac] Fix calculation of `MethodDesc` optional slot addresses (#110491)
elinor-fung Dec 10, 2024
c3e071c
[dac] Make `GetObjectStringData` return the needed buffer element cou…
elinor-fung Dec 10, 2024
7b1214b
Remove ld_classic in 16+ (#110542)
agocke Dec 10, 2024
43c5fac
Fix AV error in DAC on Linux/MacOS - issue #109877 (#110557)
mikem8361 Dec 10, 2024
40014b6
[browser] fix code gen overflow (#110539)
pavelsavara Dec 10, 2024
5d539a2
Disable `HybridGlobalization` tests for WASM on CI (#110526)
ilonatommy Dec 10, 2024
0bb7432
JIT: Include more edges in `BlockDominancePreds` (#110531)
jakobbotsch Dec 10, 2024
e0f70cc
[mono][interp] Remove no_inlining functionality for dead bblocks (#11…
BrzVlad Dec 10, 2024
bbe9a9d
Enable more ILLinker skipped tests on native AOT (#110353)
MichalStrehovsky Dec 10, 2024
5750272
JIT: Avoid comparing regnums in `GenTreeHWIntrinsic::Equals` (#110535)
jakobbotsch Dec 10, 2024
2071313
fix FastOpen compilation (#110561)
wfurt Dec 10, 2024
f430ffa
Remove duplicate IsAscii check in string.IsNormalized (#110576)
MihaZupan Dec 10, 2024
c7fc667
Remove Helper Method Frames (HMF) from Reflection (#110481)
AaronRobinsonMSFT Dec 10, 2024
df0eaa2
Speed up surrogate validation in string.Normalize (#110574)
MihaZupan Dec 10, 2024
836b868
Use holding thread id in AwareLock to avoid orphaned lock crash (#107…
eduardo-vp Dec 10, 2024
6d18e0d
Cleanup some dead code (#110579)
huoyaoyuan Dec 11, 2024
2579b1e
Revert "[browser] fix code gen overflow (#110539)" (#110599)
jkotas Dec 11, 2024
8708c3d
Share threadpool configuration (#110469)
MichalStrehovsky Dec 11, 2024
d564cb3
Ensure that we don't try and optimize masks for promoted fields (#110…
tannergooding Dec 11, 2024
22001f7
[cDAC] SOSDacImpl::GetMethodDescData DynamicMethodObject (#110545)
max-charlamb Dec 11, 2024
ff171c4
Fix comments in AggregateException.GetBaseException() (#107743)
epsitec Dec 11, 2024
9652163
[browser] Remove WASM `HybridGlobalization` from library tests, WBT a…
ilonatommy Dec 11, 2024
d97abb1
fix wrong arguments order in CrlCacheExpired call (#110457)
Alex4414 Dec 11, 2024
fe9a96a
Fix TensorExtensions.StdDev (#110392)
lilinus Dec 11, 2024
0181b15
Use FLS detach as thread termination notification on windows. (#110589)
VSadov Dec 11, 2024
2692fc5
[Profiler] Avoid Recursive ThreadStoreLock in Profiling Thread Enumer…
mdh1418 Dec 11, 2024
097ed73
Remove HttpMetricsEnrichmentContext caching (#110580)
MihaZupan Dec 11, 2024
ab2fa84
[NRBF] Reduce the most time-consuming test case to avoid timeouts for…
adamsitnik Dec 12, 2024
ae492ef
All `WasmBuildTests` use static project from assets or `dotnet new`, …
ilonatommy Dec 12, 2024
8c80358
[Mono]: Update Mono diagnostic docs. (#110621)
lateralusX Dec 12, 2024
124986b
Update dependencies from dotnet/roslyn (#110105)
am11 Dec 12, 2024
b0f79a4
[Mono]: Fix Mono profiler EventPipe provider instrumentation feature.…
lateralusX Dec 12, 2024
add0aa3
SPMI: Avoid duplicate example diffs in diffs summary (#110619)
jakobbotsch Dec 12, 2024
0c0281e
[browser] fix code gen overflow - reapply (#110606)
pavelsavara Dec 12, 2024
432af20
JIT: Remove `VisitLoopBlocksLexical` utility (#110490)
amanasifkhalid Dec 12, 2024
80b8de7
Fix crash when pTargetMD is null (#110650)
thaystg Dec 12, 2024
57e0e9c
[main] Update dependencies from dotnet/roslyn (#110084)
dotnet-maestro[bot] Dec 12, 2024
c0e3f59
JIT: Fix reporting of tier name metadata (#110610)
jakobbotsch Dec 12, 2024
b3d059f
More WriteGather fixes (#109826)
adamsitnik Dec 12, 2024
c39d942
[cdac] Always re-read global pointers in GetUsefulGlobals (#110633)
elinor-fung Dec 12, 2024
3955bc8
Correct arm64 SignExtension (#110635)
mikelle-rogers Dec 12, 2024
46946fe
Delete .GuardCF library build (#110671)
MichalStrehovsky Dec 13, 2024
32acefa
[browser] NativeAOT-LLVM support in browser-bench (#110611)
maraf Dec 13, 2024
cb8d141
JIT: Remove always-true `fgCanRelocateEHRegions` (#110612)
jakobbotsch Dec 13, 2024
a4ca48f
[wasm] Add bench output log, to the file and to the console (#110669)
radekdoulik Dec 13, 2024
34cf5bc
JIT: Add an "init BB" invariant (#110404)
jakobbotsch Dec 13, 2024
d7cc790
Remove FabricBot from area-owners.md (#110525)
akoeplinger Dec 13, 2024
15e01d4
JIT: Spill newarr into temp (#110518)
hez2010 Dec 13, 2024
07e85b6
[cdac] Fix ISOSDacInterface13.TraverseLoaderHeap parameter type (#110…
elinor-fung Dec 13, 2024
05d687e
[cdac] Handle non-IL method descs in `RuntimeTypeSystem_1.GetMethodCl…
elinor-fung Dec 13, 2024
97f8570
[cDAC] Implement GCCover portion of SOSDacImpl::GetMethodDescData (#1…
max-charlamb Dec 13, 2024
f1b1f3d
Remove unused Precode::IsCorrectMethodDesc (#110703)
elinor-fung Dec 13, 2024
1502947
JIT: capture class types when spilling a GDV arg (#110675)
AndyAyersMS Dec 14, 2024
fd3c397
[cdac] Clear cached data as part of IXCLRDataProcess::Flush (#110700)
elinor-fung Dec 14, 2024
f52248f
Improve codegen for Vector512.ExtractMostSignificatBits (#110662)
tannergooding Dec 14, 2024
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
The table of contents is too big for display.
Diff view
Diff view
  •  
  •  
  •  
2 changes: 1 addition & 1 deletion docs/area-owners.md
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ This list is for this **dotnet/runtime** repo. The **dotnet/aspnetcore** repo ha

## Areas

Note: Editing this file doesn't update the mapping used by `@msftbot` for area-specific issue/PR notifications. That configuration is part of the [`fabricbot.json`](../.github/fabricbot.json) file, and many areas use GitHub teams for those notifications. If you're a community member interested in receiving area-specific issue/PR notifications, you won't appear in this table or be added to those GitHub teams, but you can create a PR that updates `fabricbot.json` to add yourself to those notifications. See [automation.md](infra/automation.md) for more information on the schema and tools used by FabricBot.
Note: Editing this file doesn't update the mapping used by `@dotnet-policy-service` for area-specific issue/PR notifications. That configuration is part of the [`resourceManagement.yml`](../.github/policies/resourceManagement.yml) file, and many areas use GitHub teams for those notifications. If you're a community member interested in receiving area-specific issue/PR notifications, you won't appear in this table or be added to those GitHub teams, but you can create a PR that updates `resourceManagement.yml` to add yourself to those notifications. See [automation.md](infra/automation.md) for more information.

| Area | Lead | Owners (area experts to tag in PRs and issues) | Notes |
|------------------------------------------------|----------------------|------------------------------------------------------------------------------------------|-------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------|
Expand Down
10 changes: 5 additions & 5 deletions docs/design/coreclr/botr/profiling.md
Original file line number Diff line number Diff line change
Expand Up @@ -68,10 +68,10 @@ The other interface involved for profiling is _ICorProfilerInfo_. The profiler

The picture so far describes what happens once the application and profiler are running. But how are the two connected together when an application is started? The CLR makes the connection during its initialization in each process. It decides whether to connect to a profiler, and which profiler that should be, depending upon the value for two environment variables, checked one after the other:

- Cor\_Enable\_Profiling - only connect with a profiler if this environment variable exists and is set to a non-zero value.
- Cor\_Profiler - connect with the profiler with this CLSID or ProgID (which must have been stored previously in the Registry). The Cor\_Profiler environment variable is defined as a string:
- set Cor\_Profiler={32E2F4DA-1BEA-47ea-88F9-C5DAF691C94A}, or
- set Cor\_Profiler="MyProfiler"
- CORECLR\_ENABLE\_PROFILING - only connect with a profiler if this environment variable exists and is set to a non-zero value.
- CORECLR\_PROFILER - connect with the profiler with this CLSID or ProgID (which must have been stored previously in the Registry). The CORECLR\_PROFILER environment variable is defined as a string:
- set CORECLR\_PROFILER={32E2F4DA-1BEA-47ea-88F9-C5DAF691C94A}, or
- set CORECLR\_PROFILER="MyProfiler"
- The profiler class is the one that implements _ICorProfilerCallback/ICorProfilerCallback2_. It is required that a profiler implement ICorProfilerCallback2; if it does not, it will not be loaded.

When both checks above pass, the CLR creates an instance of the profiler in a similar fashion to _CoCreateInstance_. The profiler is not loaded through a direct call to _CoCreateInstance_ so that a call to _CoInitialize_ may be avoided, which requires setting the threading model. It then calls the _ICorProfilerCallback::Initialize_ method in the profiler. The signature of this method is:
Expand Down Expand Up @@ -235,7 +235,7 @@ Profiling is enabled through environment variables, and since NT Services are st

MyComputer -> Properties -> Advanced -> EnvironmentVariables -> System Variables

Both **Cor\_Enable\_Profiling** and **COR\_PROFILER have to be set** , and the user must ensure that the Profiler DLL is registered. Then, the target machine should be re-booted so that the NT Services pick up those changes. Note that this will enable profiling on a system-wide basis. So, to prevent every managed application that is run subsequently from being profiled, the user should delete those system environment variables after the re-boot.
Both **CORECLR\_ENABLE\_PROFILING** and **CORECLR\_PROFILER have to be set** , and the user must ensure that the Profiler DLL is registered. Then, the target machine should be re-booted so that the NT Services pick up those changes. Note that this will enable profiling on a system-wide basis. So, to prevent every managed application that is run subsequently from being profiled, the user should delete those system environment variables after the re-boot.

Profiling API – High-Level Description
======================================
Expand Down
13 changes: 13 additions & 0 deletions docs/design/datacontracts/CodeVersions.md
Original file line number Diff line number Diff line change
Expand Up @@ -40,6 +40,9 @@ public virtual bool CodeVersionManagerSupportsMethod(TargetPointer methodDesc);

// Return the instruction pointer corresponding to the start of the given native code version
public virtual TargetCodePointer GetNativeCode(NativeCodeVersionHandle codeVersionHandle);

// Gets the GCStressCodeCopy pointer if available, otherwise returns TargetPointer.Null
public virtual TargetPointer GetGCStressCodeCopy(NativeCodeVersionHandle codeVersionHandle);
```
### Extension Methods
```csharp
Expand All @@ -61,13 +64,15 @@ Data descriptors used:
| NativeCodeVersionNode | NativeCode | indicates an explicit native code version node |
| NativeCodeVersionNode | Flags | `NativeCodeVersionNodeFlags` flags, see below |
| NativeCodeVersionNode | VersionId | Version ID corresponding to the parent IL code version |
| NativeCodeVersionNode | GCCoverageInfo | GCStress debug info, if supported |
| ILCodeVersioningState | FirstVersionNode | pointer to the first `ILCodeVersionNode` |
| ILCodeVersioningState | ActiveVersionKind | an `ILCodeVersionKind` value indicating which fields of the active version are value |
| ILCodeVersioningState | ActiveVersionNode | if the active version is explicit, the NativeCodeVersionNode for the active version |
| ILCodeVersioningState | ActiveVersionModule | if the active version is synthetic or unknown, the pointer to the Module that defines the method |
| ILCodeVersioningState | ActiveVersionMethodDef | if the active version is synthetic or unknown, the MethodDef token for the method |
| ILCodeVersionNode | VersionId | Version ID of the node |
| ILCodeVersionNode | Next | Pointer to the next `ILCodeVersionNode`|
| GCCoverageInfo | SavedCode | Pointer to the GCCover saved code copy, if supported |

The flag indicates that the default version of the code for a method desc is active:
```csharp
Expand Down Expand Up @@ -249,3 +254,11 @@ bool ICodeVersions.CodeVersionManagerSupportsMethod(TargetPointer methodDescAddr
return true;
}
```

### Finding GCStress Code Copy
```csharp
public virtual TargetPointer GetGCStressCodeCopy(NativeCodeVersionHandle codeVersionHandle);
```

1. If `codeVersionHandle` is synthetic, use the `IRuntimeTypeSystem` to find the GCStressCodeCopy.
2. If `codeVersionHandle` is explicit, read the `NativeCodeVersionNode` for the `GCCoverageInfo` pointer. This value only exists in some builds. If the value doesn't exist or is a nullptr, return `TargetPointer.Null`. Otherwise return the `SavedCode` pointer from the `GCCoverageInfo` struct.
4 changes: 4 additions & 0 deletions docs/design/datacontracts/RuntimeTypeSystem.md
Original file line number Diff line number Diff line change
Expand Up @@ -159,6 +159,8 @@ partial interface IRuntimeTypeSystem : IContract
// Get an instruction pointer that can be called to cause the MethodDesc to be executed
public virtual TargetCodePointer GetNativeCode(MethodDescHandle methodDesc);

// Gets the GCStressCodeCopy pointer if available, otherwise returns TargetPointer.Null
public virtual TargetPointer GetGCStressCodeCopy(MethodDescHandle methodDesc);
}
```

Expand Down Expand Up @@ -638,6 +640,7 @@ We depend on the following data descriptors:
| `MethodDesc` | `Slot` | The method's slot |
| `MethodDesc` | `Flags` | The method's flags |
| `MethodDesc` | `Flags3AndTokenRemainder` | More flags for the method, and the low bits of the method's token's RID |
| `MethodDesc` | `GCCoverageInfo` | The method's GCCover debug info, if supported |
| `MethodDescCodeData` | `VersioningState` | The IL versioning state associated with a method descriptor
| `MethodDescChunk` | `MethodTable` | The method table set of methods belongs to |
| `MethodDescChunk` | `Next` | The next chunk of methods |
Expand All @@ -654,6 +657,7 @@ We depend on the following data descriptors:
| `StoredSigMethodDesc` | `cSig` | Count of bytes in the metadata signature |
| `StoredSigMethodDesc` | `ExtendedFlags` | Flags field for the `StoredSigMethodDesc` |
| `DynamicMethodDesc` | `MethodName` | Pointer to Null-terminated UTF8 string describing the Method desc |
| `GCCoverageInfo` | `SavedCode` | Pointer to the GCCover saved code copy, if supported |


The contract depends on the following other contracts
Expand Down
Loading
Loading