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

Allow storage classes other than static #343

Open
ForNeVeR opened this issue Oct 23, 2022 · 0 comments
Open

Allow storage classes other than static #343

ForNeVeR opened this issue Oct 23, 2022 · 0 comments
Labels
area:compiler Related to code compilation or type checking good-first-issue An issue considered simple enough for new contributors kind:feature New feature or request status:help-wanted Open for contributors

Comments

@ForNeVeR
Copy link
Owner

ForNeVeR commented Oct 23, 2022 •
edited
Loading

C standard (6.7.1 Storage-class specifiers) defines the following storage classes:

  • typedef (not a real storage class)
  • extern
  • static
  • _Thread_local
  • auto
  • register

Only one storage class per declaration is allowed (with an exception of _Thread_local with either static or extern).

We should allow these storage classes in the Cesium.CodeGen.Ir.Declarations.ScopedIdentifierDeclaration.

When implementing this, find the number 343 in the sources.

This issue doesn't include full processing of these storage classes, but only allowing them in the IR classes and proper parsing.
@ForNeVeR ForNeVeR added kind:feature New feature or request status:help-wanted Open for contributors area:compiler Related to code compilation or type checking good-first-issue An issue considered simple enough for new contributors labels Oct 23, 2022
ForNeVeR added a commit that referenced this issue Oct 23, 2022
@ForNeVeR
(#342, #343) IR: register the TODOs
5394b49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area:compiler Related to code compilation or type checking good-first-issue An issue considered simple enough for new contributors kind:feature New feature or request status:help-wanted Open for contributors
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@ForNeVeR