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

Separate dx12 codegen #1827

Merged
merged 1 commit into from
Oct 28, 2024

Conversation

MarkY-LunarG
Copy link
Contributor

Separate out the DX12 codegen from the Vulkan codegen. The DX12 codegen was heavily tied in with the Vulkan codegen which leads to difficulty when trying to add in other Khronos API codegen (since we have to be careful of breaking other APIs).

@ci-tester-lunarg
Copy link

CI gfxreconstruct build queued with queue ID 285599.

@ci-tester-lunarg
Copy link

CI gfxreconstruct build # 5166 running.

@ci-tester-lunarg
Copy link

CI gfxreconstruct build # 5166 passed.

@ci-tester-lunarg
Copy link

CI gfxreconstruct build queued with queue ID 286429.

@ci-tester-lunarg
Copy link

CI gfxreconstruct build # 5180 running.

@ci-tester-lunarg
Copy link

CI gfxreconstruct build # 5180 passed.

@bradgrantham-lunarg bradgrantham-lunarg removed their request for review October 24, 2024 21:52
@bradgrantham-lunarg
Copy link
Contributor

Removed myself from reviewers in favor of @davidd-lunarg

@ci-tester-lunarg
Copy link

CI gfxreconstruct build # 5166 passed.

@MarkY-LunarG
Copy link
Contributor Author

Reverted yapf change.

@ci-tester-lunarg
Copy link

CI gfxreconstruct build queued with queue ID 288060.

@ci-tester-lunarg
Copy link

CI gfxreconstruct build # 5201 running.

@ci-tester-lunarg
Copy link

CI gfxreconstruct build queued with queue ID 288080.

@ci-tester-lunarg
Copy link

CI gfxreconstruct build # 5202 running.

@ci-tester-lunarg
Copy link

CI gfxreconstruct build # 5202 passed.

@ci-tester-lunarg
Copy link

CI gfxreconstruct build queued with queue ID 288128.

@ci-tester-lunarg
Copy link

CI gfxreconstruct build # 5203 running.

@ci-tester-lunarg
Copy link

CI gfxreconstruct build # 5203 passed.

Remove the hard-coded dependency of the DX12 GFXR codegen upon our
Vulkan and the Khronos registry edit functionality (which the DX
side really doesn't use).
This includes removing the dependency on the "base_generators"
directory as well.
NOTE: This did require copying a functionality (like write) from
the Khronos source to maintain consistency (so I added the Khronos
copyright notice to the dx12_generators/base_generator.py file).

The reason behind this is there are big changes coming to the Vulkan
side in preparation of supporting other Khronos APIs and these changes
could break DX12.
@MarkY-LunarG
Copy link
Contributor Author

Squashing for merge, once it passes I will merge.

@ci-tester-lunarg
Copy link

CI gfxreconstruct build queued with queue ID 289377.

@ci-tester-lunarg
Copy link

CI gfxreconstruct build # 5216 running.

@ci-tester-lunarg
Copy link

CI gfxreconstruct build # 5216 passed.

@MarkY-LunarG MarkY-LunarG merged commit 6694f65 into LunarG:dev Oct 28, 2024
9 checks passed
@MarkY-LunarG MarkY-LunarG deleted the marky_separate_dx12_codegen branch October 28, 2024 17:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants