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

DR and SR duplication and merging #111

Open
Skepfyr opened this issue Oct 7, 2019 · 1 comment
Open

DR and SR duplication and merging #111

Skepfyr opened this issue Oct 7, 2019 · 1 comment
Labels

Comments

@Skepfyr
Copy link
Contributor

Skepfyr commented Oct 7, 2019

Currently, SR is built on top of DR, as DR is already written and it provides a useful and productive base to work on. However, SR may be simpler as a parser from binary blob form, so if SR is requiring large scale changes in DR code then it is worthwhile considering merging the two at that point to reduce duplicated effort.

@Skepfyr Skepfyr added the tracker label Oct 7, 2019
@kvark
Copy link
Member

kvark commented Oct 8, 2019

I think DR has the following values today:

  1. it's good for bootstrapping SR
  2. it's fairly straightforward and sound in what it tries to do

However, when SR is there I don't think we'll see anybody wanting to use DR specifically as opposed to either raw spirv or SR. So in the long run we should probably just have just SR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants