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.

Sign up for GitHub

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

Jump to bottom

Need to delay type loading error until runtime for array of T with ByRefLike type #99165

Closed
Tracked by #94415
fanyang-mono opened this issue Mar 1, 2024 · 1 comment · Fixed by #99081
Closed
Tracked by #94415

Need to delay type loading error until runtime for array of T with ByRefLike type #99165

fanyang-mono opened this issue Mar 1, 2024 · 1 comment · Fixed by #99081
Assignees
@fanyang-mono
Labels
area-VM-meta-mono disabled-test The test is disabled in source code against the issue
Milestone
9.0.0

Comments

@fanyang-mono
Copy link
Member

When uncommenting AllocArrayOfT_Invalid and AllocMultiDimArrayOfT_Invalid in InvalidCSharp.il, AOT compile Validate.dll triggered the following assertion. Need to delay this error untill runtime.

  aot-compile: compiling /__w/1/s/artifacts/tests/coreclr/linux.x64.Release/Loader/classloader/generics/ByRefLike/Validate/Validate.dll; MONO_PATH: /__w/1/s/artifacts/tests/coreclr/linux.x64.Release/Loader/classloader/generics/ByRefLike/Validate:/__w/1/s/artifacts/tests/coreclr/linux.x64.Release/Tests/Core_Root
  Mono Ahead of Time compiler - compiling assembly /__w/1/s/artifacts/tests/coreclr/linux.x64.Release/Loader/classloader/generics/ByRefLike/Validate/Validate.dll
  AOTID E90E991F-9074-7937-4FB1-E9B7D16A073B
  * Assertion at /__w/1/s/src/mono/mono/metadata/loader.c:826, condition `!mono_class_has_failure (klass)' not met
@fanyang-mono fanyang-mono added this to the 9.0.0 milestone Mar 1, 2024
@fanyang-mono fanyang-mono self-assigned this Mar 1, 2024
@fanyang-mono fanyang-mono mentioned this issue Mar 1, 2024
Closed
5 tasks
@fanyang-mono fanyang-mono added the disabled-test The test is disabled in source code against the issue label Mar 1, 2024
@fanyang-mono fanyang-mono mentioned this issue Mar 11, 2024
Merged
@fanyang-mono
Copy link
Member Author

This issue has been addressed by #99081.

@github-actions github-actions bot locked and limited conversation to collaborators Apr 13, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@fanyang-mono fanyang-mono

Labels
area-VM-meta-mono disabled-test The test is disabled in source code against the issue
Projects
None yet
Milestone
9.0.0
Development

Successfully merging a pull request may close this issue.

[mono] Add basic ref struct support for generic parameter fanyang-mono/runtime
1 participant
@fanyang-mono