-
Notifications
You must be signed in to change notification settings - Fork 4.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Use separate VNs for DBL2INT_OVF helpers #58868
Conversation
Tagging subscribers to this area: @JulieLeeMSFT Issue DetailsFixes #58832 I don't think it makes much sense to share VNs between OVF and non-OVF helpers in terms of performance. /cc @SingleAccretion feel free to rewrite it to VNForCast for 7.0 SPMI-diffs (only caught coreclr_tests.pmi.windows.x64.checked.mch:
Detail diffs
|
CC @AndyAyersMS please review this PR targetting .NET6 port. |
/backport to release/6.0 |
Started backporting to release/6.0: https://github.com/dotnet/runtime/actions/runs/1220945364 |
Fixes #58832
I don't think it makes much sense to share VNs between OVF and non-OVF helpers in terms of performance.
/cc @SingleAccretion feel free to rewrite it to VNForCast for 7.0
SPMI-diffs (only caught
ValueNumberingCheckedCastsOfConstants
test where such cases are tested):coreclr_tests.pmi.windows.x64.checked.mch:
Detail diffs