Skip to content
This repository was archived by the owner on Jan 23, 2023. It is now read-only.

Replacing usages of random with TestLibrary.Generator in the HWIntrinsic templated tests #19062

Merged
merged 6 commits into from
Jul 24, 2018
Merged

Conversation

tannergooding
Copy link
Member

FYI. @CarolEidt, @fiigii, @echesakovMSFT

@tannergooding
Copy link
Member Author

Tried to break this up into logical commits. There are still some usages of random, in the non-templated tests. I am working on moving those to be templated.

@CarolEidt
Copy link

Tried to break this up into logical commits. There are still some usages of random, in the non-templated tests. I am working on moving those to be templated.

Thanks! that makes it much easier to review.

Copy link

@CarolEidt CarolEidt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@tannergooding
Copy link
Member Author

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.

@tannergooding
Copy link
Member Author

Rebased onto head of master, as I can't repro the JitStress failures locally.

@tannergooding
Copy link
Member Author

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 -priority=1

@tannergooding
Copy link
Member Author

@dotnet-bot test Windows_NT x64 Checked jitincompletehwintrinsic
@dotnet-bot test Windows_NT x64 Checked jitx86hwintrinsicnoavx
@dotnet-bot test Windows_NT x64 Checked jitx86hwintrinsicnoavx2
@dotnet-bot test Windows_NT x64 Checked jitx86hwintrinsicnosimd
@dotnet-bot test Windows_NT x64 Checked jitnox86hwintrinsic

@dotnet-bot test Windows_NT x86 Checked jitincompletehwintrinsic
@dotnet-bot test Windows_NT x86 Checked jitx86hwintrinsicnoavx
@dotnet-bot test Windows_NT x86 Checked jitx86hwintrinsicnoavx2
@dotnet-bot test Windows_NT x86 Checked jitx86hwintrinsicnosimd
@dotnet-bot test Windows_NT x86 Checked jitnox86hwintrinsic

@dotnet-bot test Ubuntu x64 Checked jitincompletehwintrinsic
@dotnet-bot test Ubuntu x64 Checked jitx86hwintrinsicnoavx
@dotnet-bot test Ubuntu x64 Checked jitx86hwintrinsicnoavx2
@dotnet-bot test Ubuntu x64 Checked jitx86hwintrinsicnosimd
@dotnet-bot test Ubuntu x64 Checked jitnox86hwintrinsic

@tannergooding
Copy link
Member Author

Rebased onto dotnet/master now that the JitStress issue has been resolved.

@tannergooding tannergooding merged commit 2b92e13 into dotnet:master Jul 24, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants