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

Use fully qualified names in spec #108

Open
misyltoad opened this issue Apr 27, 2021 · 1 comment
Open

Use fully qualified names in spec #108

misyltoad opened this issue Apr 27, 2021 · 1 comment

Comments

@misyltoad
Copy link

A problem I routinely have when trying to read the SPIR-V spec is that finding things is a chore as some things have very generic names like Offset, R or 2D, etc which have multiple references in the spec.

I think it would be better to use fully qualified names everywhere such as ImageOperandsOffsetMask, ImageChannelOrderR, Dim2D

This is not just a problem for these specific names and enums, this problem appears for a huge amount of names in the SPIR-V specification and makes it really hard to search for things.
@raunraun
Copy link
Contributor

raunraun commented Jul 7, 2021

Discussed in the SPIR working group on 6/30/2020

  • We're taking a look
  • 91 occurrences of 'offset' seems like something worth taking a look at.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@raunraun @misyltoad