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

[SR-6889] [IRGen Opportunity] Use argmemonly for runtime functions #49438

Open
aschwaighofer opened this issue Feb 1, 2018 · 0 comments
Open
Labels
compiler The Swift compiler in itself good first issue Good for newcomers improvement

Comments

@aschwaighofer
Copy link
Member

Previous ID SR-6889
Radar None
Original Reporter @aschwaighofer
Type Improvement
Additional Detail from JIRA
Votes 0
Component/s Compiler
Labels Improvement, StarterBug
Assignee None
Priority Medium

md5: 050d922fc6f6b8f37d1f70d51229bf7b

Issue Description:

It should be possible to mark some of our runtime functions with argmemonly readonly as they receive their type arguments indirectly via an array of type arguments.

I tried this in #14310 but tests failed. I did not have time to investigate why.

I mentioned that if that following situation in some form or another this annotation is not legal:

Type *args[5];
pointer->field = // inlined initialization
arg[1] = pointer
argmemonly(args)

However, we believe that this situation should? not arise. It might be worth investigating the failures.

@swift-ci swift-ci transferred this issue from apple/swift-issues Apr 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
compiler The Swift compiler in itself good first issue Good for newcomers improvement
Projects
None yet
Development

No branches or pull requests

1 participant