Skip to content
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

a new poison test is failing in ci #55062

Closed
sandreenko opened this issue Jul 2, 2021 · 4 comments · Fixed by #56149
Closed

a new poison test is failing in ci #55062

sandreenko opened this issue Jul 2, 2021 · 4 comments · Fixed by #56149
Assignees
Labels
area-CodeGen-coreclr CLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMI disabled-test The test is disabled in source code against the issue
Milestone

Comments

@sandreenko
Copy link
Contributor

Examples https://dev.azure.com/dnceng/public/_build/results?buildId=1216832&view=ms.vss-test-web.build-test-results-tab
added in #54685

@sandreenko sandreenko added disabled-test The test is disabled in source code against the issue area-CodeGen-coreclr CLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMI labels Jul 2, 2021
@sandreenko sandreenko added this to the 6.0.0 milestone Jul 2, 2021
@dotnet-issue-labeler dotnet-issue-labeler bot added the untriaged New issue has not been triaged by the area owner label Jul 2, 2021
@jakobbotsch
Copy link
Member

This seems to be failing because crossgen2 does not pass the CORJIT_FLAG_DEBUG_CODE to the JIT when optimizing, even when the module has the debug flag set. The crossgen jobs all pass -O to crossgen, so poisoning does not get turned on for the test.

Along with poisoning the flag also makes the JIT extend lifetimes on refs, ensure all IL basic blocks emit some code that can be breakpointed and ensure exception throws get unique code sites. The question is if crossgen2 should respect the debug flag in the module even when -O is passed. Thoughts @dotnet/crossgen-contrib?

@mangod9
Copy link
Member

mangod9 commented Jul 2, 2021

Yeah looks like it cg2 doesnt check the debuggable attribute if -O is explicitly specified. Its always been this way though, are you noticing that this started failing recently?

// In single-file compilation mode, use the assembly's DebuggableAttribute to determine whether to optimize

@jakobbotsch
Copy link
Member

No, the test is new. So the alternative is to disable this test in crossgen2 jobs.

@mangod9
Copy link
Member

mangod9 commented Jul 3, 2021

we should fix it in cg2, but if its blocking yeah please disable for r2r jobs till its fixed.

@jeffschwMSFT jeffschwMSFT removed the untriaged New issue has not been triaged by the area owner label Jul 6, 2021
jakobbotsch added a commit to jakobbotsch/runtime that referenced this issue Jul 22, 2021
@ghost ghost added the in-pr There is an active PR which will close this issue when it is merged label Jul 22, 2021
@ghost ghost removed the in-pr There is an active PR which will close this issue when it is merged label Jul 23, 2021
@ghost ghost locked as resolved and limited conversation to collaborators Aug 22, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-CodeGen-coreclr CLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMI disabled-test The test is disabled in source code against the issue
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants