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

[x86/Linux] Port 'FaultingExceptionFrame::UpdateRegDisplay' #8993

Conversation

parjong
Copy link

@parjong parjong commented Jan 18, 2017

This was a part of #8981, but seems to be necessary to enable x86/Linux exception handling.

@parjong
Copy link
Author

parjong commented Jan 18, 2017

\CC @seanshpark

@parjong
Copy link
Author

parjong commented Jan 19, 2017

@janvorli The following tests failed with this PR (You could see full log here)

  • GC.Regressions_v2_0_beta1_149926_149926_149926._Regressions_v2_0_beta1_149926_149926_149926_cmd
  • JIT.Regression_CLR_x86_JIT_V1_M12_Beta2_b91377_b91377_b91377.Regression_CLR_x86_JIT_V1_M12 - Beta2_b91377_b91377_b91377_cmd
  • JIT.jit64_opt_cse_HugeField2_HugeField2._jit64_opt_cse_HugeField2_HugeField2_cmd

Is it possible that this PR affects x64/Windows port?, or are there any known issues related with these tests?

@parjong
Copy link
Author

parjong commented Jan 19, 2017

@dotnet-bot test Windows_NT x64 Debug Build and Test please

@janvorli
Copy link
Member

@parjong no, the change code is purely x86 code and it is not used on other architectures. But I have checked the recent test jobs and I can see that few semi-random tests in the Windows NT run fail intermittently, so it is probably the case here.

@parjong
Copy link
Author

parjong commented Jan 19, 2017

@janvorli Thank you for confirm!

@parjong
Copy link
Author

parjong commented Jan 19, 2017

@dotnet-bot test Windows_NT x64 Debug Build and Test please

Copy link
Member

@janvorli janvorli left a comment

Choose a reason for hiding this comment

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

LGTM, thank you!

@janvorli janvorli merged commit 0d04afc into dotnet:master Jan 19, 2017
sergign60 pushed a commit to sergign60/coreclr that referenced this pull request Jan 19, 2017
@parjong parjong deleted the fix/x86_FaultingExceptionFrame_UpdateRegDisplay branch January 20, 2017 00:17
@karelz karelz modified the milestone: 2.0.0 Aug 28, 2017
picenka21 pushed a commit to picenka21/runtime that referenced this pull request Feb 18, 2022
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.

4 participants