-
Notifications
You must be signed in to change notification settings - Fork 2.6k
Conversation
Also add to the arm & arm64 tests.lst
Need to build a use for each reg. Also, dump the defList if it's not empty at end of block.
The first commit is the test; it fails as expected on arm (it doesn't fail with the altjit). |
The problem was that LSRA was creating only a singe use for multi-reg ops under FIELD_LIST. We need to build a use for each reg. |
@dotnet/jit-contrib PTAL |
Link: #18362 |
Fix #18362 |
@BruceForstall @jashook the reason this didn't repro with the altjit is that |
I will let Bruce comment further; however, adding in altjit specific return values to me sounds dangerous. |
@CarolEidt I think we should make it work for altjit as you suggest. Wrap |
Fix & test for dotnet/coreclr#18362 Commit migrated from dotnet/coreclr@13e7606
Also add to the arm & arm64 tests.lst