-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
Some ASP.NET Core 10.0 dlls are not R2R compiled #111997
Comments
I am not aware of any. Do you see crossgen2 running on these assemblies in your build log? Does it produce any errors? |
I don't see any errors in the log, and I found:
I took a look in the aspnetcore repo, and I didn't find a reason why this project would be treated differently from the others (which are passing the R2R check). I'll do a local build so I can see if any |
@tmds are you sure that the assembly has functions to prejit? if it's only a bunch of interfaces/facades, then presumably there is no work for R2R. From a quick look, |
crossgen2 will produce R2R image even if there is no code to compile. R2R image has other good stuff in addition to the code, for example indexes for some metadata tables. |
I had also assumed something might have changed which made crossgen2 skip these assemblies which is why I created the issue in this repo. Looking at the assemblies produced during the vmr aspnetcore build, there is a crossgenned There isn't one at I'm going to close this issue and create one in aspnetcore repo instead. |
aspnetcore issue: dotnet/aspnetcore#60174. |
As part of our test suite, we have a test that checks .NET's dlls are R2R compiled.
That test is failing for these ASP.NET Core 10.0 dlls:
To determine whether the assembly is R2R we check
PEHeaders.CorHeader.ManagedNativeHeaderDirectory.Size != 0
(cfr #1802 (comment)).The check is passing for the other ASP.NET Core dlls. The two dlls that fail don't have much in them (some interface definitions).
I wonder if there has been a change to crossgen which would causes these dlls to be skipped for R2R?
The check passes for the 9.0 version of these dlls.
cc @jkoritzinsky @jkotas @omajid @nicrowe00
The text was updated successfully, but these errors were encountered: