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

[Tracking] CoreCLR support for InlineArrayAttribute #81145

Closed
2 of 3 tasks
AaronRobinsonMSFT opened this issue Jan 25, 2023 · 7 comments
Closed
2 of 3 tasks

[Tracking] CoreCLR support for InlineArrayAttribute #81145

AaronRobinsonMSFT opened this issue Jan 25, 2023 · 7 comments
Assignees
Labels
area-VM-coreclr tracking This issue is tracking the completion of other related issues.
Milestone

Comments

@AaronRobinsonMSFT
Copy link
Member

AaronRobinsonMSFT commented Jan 25, 2023

This issue tracks the work for implementing CoreCLR support for InlineArrayAttribute #61135 and the related C# features such as params ReadOnlySpan, safe fixed size buffers, stackalloc of reference types, etc.

Required work:

@AaronRobinsonMSFT AaronRobinsonMSFT added this to the 8.0.0 milestone Jan 25, 2023
@AaronRobinsonMSFT AaronRobinsonMSFT added the tracking This issue is tracking the completion of other related issues. label Jan 25, 2023
@AaronRobinsonMSFT
Copy link
Member Author

This follows from the Mono tracking issue #80798

/cc @mangod9

@AaronRobinsonMSFT
Copy link
Member Author

/cc @VSadov

@mangod9
Copy link
Member

mangod9 commented Jul 29, 2023

@VSadov assume this is ok to close?

@AaronRobinsonMSFT
Copy link
Member Author

@VSadov assume this is ok to close?

I don't think either of the two open check boxes are complete.

@mangod9
Copy link
Member

mangod9 commented Aug 3, 2023

@VSadov, is this on your radar to finish in 8?

@VSadov
Copy link
Member

VSadov commented Aug 3, 2023

I was just looking at this. I do not think there are any showstoppers here.

The marshaling part mentions that for 8.0 it could be just a documentation matter and later we can see if more is needed.
The “implement diagnostics” is very vague. Should that be an issue on VS/debugger? Do we have diagnostics support for Span<T>, just as an example?

I think we can close this or move to 9.0 to follow up as we see more use of the feature and it becomes more clear what can be improved.

@mangod9
Copy link
Member

mangod9 commented Aug 9, 2023

Closing this based on the comment above. We can create new issues as required to track remaining work.

@mangod9 mangod9 closed this as completed Aug 9, 2023
@ghost ghost locked as resolved and limited conversation to collaborators Sep 23, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-VM-coreclr tracking This issue is tracking the completion of other related issues.
Projects
None yet
Development

No branches or pull requests

3 participants