Revert "Kill REG_PROFILER_RET_SCRATCH (#37679)"#38691
Revert "Kill REG_PROFILER_RET_SCRATCH (#37679)"#38691AndyAyersMS wants to merge 2 commits intodotnet:masterfrom
Conversation
This reverts commit b179e19.
|
Most of the stress runs are done (and all the linux arm ones) -- there are about 30 failures on linux arm amd ~100 total failures. Arm linux failures are mostly in various HW Intrinsics tests. So it seems the change I've reverted above is the root cause for the spike in failures. @BruceForstall @CarolEidt -- Seems like we should revert #37679. |
|
Agreed, although #37223 will regress and will need a different fix. It's optional that you'd want to disable the test there that will regress. |
|
What's the mechanism for excluding a libraries test? |
|
E.g., or, e.g., #38580 |
|
Can you articulate why reverting this change eliminates the issue? Codegen still uses r2, I believe, and it is codegen, not LSRA, that tracks the GC state. |
|
There are special tricks to extend gc liveness of |
|
@BruceForstall looks like we didn't specify which test hit this assert, so I don't know what to disable. It might affect more than one test. @CarolEidt do you recall? |
|
@AndyAyersMS - were you asking about which test hit the assert that I fixed with #37679? The issue was #37223 and it failed in |
|
@CarolEidt thanks -- the method was Let me see if the failure still repros and if so we can figure out whether to exclude it here or do something else. |
|
Hmm, can't get CI to run. Let me close this an open a new PR. |
This reverts commit b179e19.