-
Notifications
You must be signed in to change notification settings - Fork 5.4k
JIT: add struct promotion stress mode #8227
Copy link
Copy link
Closed
Labels
JitStressCLR JIT issues involving JIT internal stress modesCLR JIT issues involving JIT internal stress modesarea-CodeGen-coreclrCLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMICLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMIhelp wanted[up-for-grabs] Good issue for external contributors[up-for-grabs] Good issue for external contributorstenet-reliabilityReliability/stability related issue (stress, load problems, etc.)Reliability/stability related issue (stress, load problems, etc.)test-enhancementImprovements of test source codeImprovements of test source code
Milestone
Metadata
Metadata
Assignees
Labels
JitStressCLR JIT issues involving JIT internal stress modesCLR JIT issues involving JIT internal stress modesarea-CodeGen-coreclrCLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMICLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMIhelp wanted[up-for-grabs] Good issue for external contributors[up-for-grabs] Good issue for external contributorstenet-reliabilityReliability/stability related issue (stress, load problems, etc.)Reliability/stability related issue (stress, load problems, etc.)test-enhancementImprovements of test source codeImprovements of test source code
Type
Fields
Give feedbackNo fields configured for issues without a type.
Given the issues with dotnet/corefx#20025, #8185, and #8195, which were triggered by dotnet/coreclr#10453 even though it passed normal core and desktop testing, it may well be a good idea to add a stress mode to try to flush out such bugs. The stress mode would short-circuit this check so it never decides to undo a promotion.
Going further, short-circuiting
lvaShouldPromoteStructVarto always return true as part of struct promotion stress mode could help flush out more issues.category:correctness
theme:structs
skill-level:intermediate
cost:medium
impact:small