Skip to content

Added fix and regression test for primitive id on FXC#9218

Merged
teoxoy merged 1 commit intogfx-rs:trunkfrom
inner-daemons:fix-primitive-id
Mar 13, 2026
Merged

Added fix and regression test for primitive id on FXC#9218
teoxoy merged 1 commit intogfx-rs:trunkfrom
inner-daemons:fix-primitive-id

Conversation

@inner-daemons
Copy link
Copy Markdown
Collaborator

Connections
Closes #9105

Description
Explained in the code too, but FXC treats primitive ID as a non SV so it must appear before all other SVs.

No changelog entry is required because primitive ID will be added in this release.

Testing
Tested using a regression test

Squash or Rebase?
Doesn't matter

Checklist

  • Run cargo fmt.
  • Run taplo format.
  • Run cargo clippy --tests. If applicable, add:
    • --target wasm32-unknown-unknown
  • Run cargo xtask test to run tests.
  • If this contains user-facing changes, add a CHANGELOG.md entry.

Copy link
Copy Markdown
Member

@teoxoy teoxoy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good stuff!

@teoxoy teoxoy merged commit 41f7c28 into gfx-rs:trunk Mar 13, 2026
93 of 94 checks passed
@inner-daemons inner-daemons deleted the fix-primitive-id branch March 13, 2026 23:03
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.

Failures in CTS tests using primitive_index on DX12 with FXC

2 participants