-
Notifications
You must be signed in to change notification settings - Fork 2.6k
Replacing usages of random with TestLibrary.Generator in the HWIntrinsic templated tests #19062
Conversation
Tried to break this up into logical commits. There are still some usages of |
Thanks! that makes it much easier to review. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
The JitStress failures look to all be the same and are all occurring outside this PR. Going to briefly investigate to see if I can find the issue. |
Rebased onto head of master, as I can't repro the JitStress failures locally. |
JitStress jobs are failing because of #18695. I have logged https://github.com/dotnet/coreclr/issues/19075 to track. Wasn't reproing locally, initially, as it requires you to build with |
…sic template metadata
@dotnet-bot test Windows_NT x64 Checked jitincompletehwintrinsic @dotnet-bot test Windows_NT x86 Checked jitincompletehwintrinsic @dotnet-bot test Ubuntu x64 Checked jitincompletehwintrinsic |
Rebased onto dotnet/master now that the JitStress issue has been resolved. |
FYI. @CarolEidt, @fiigii, @echesakovMSFT