-
Notifications
You must be signed in to change notification settings - Fork 804
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
[RFC FS-1109] Additional intrinsics for the NativePtr module #11682
Conversation
@Happypig375 Where is this at w.r.t. completness, and the RFC? |
Just missing the RFC, writing now. |
@KevinRansom This looks good to me. Do we want "Experimental this is preview" attributes on the additions to FSharp.Core? Thanks |
@vzarytovskii Approved subject to @KevinRansom 's opinion on whether we need the "this is in preview" library attributes. |
No I think the preview attributes are not necessary. The APIs are not likely to change, they don't support a specific language feature and Dev 17.0 is the target for main anyway. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks great
Cool thank you! So we have our first non-preview F# 5.1 (or F# 6.0 or whatever) feature :) |
Thank you @Happypig375 :) |
5.1 I think. |
RFC https://github.com/fsharp/fslang-design/blob/main/preview/FS-1109-Additional-intrinsics-for-the-NativePtr-module.md
fsharp/fslang-suggestions#200