Skip to content

Isolate WGSL-specific concepts governing diagnostic filters#6503

Closed
ErichDonGubler wants to merge 2 commits intogfx-rs:trunkfrom
erichdongubler-mozilla:distinguish-diag-filters-and-wgsl
Closed

Isolate WGSL-specific concepts governing diagnostic filters#6503
ErichDonGubler wants to merge 2 commits intogfx-rs:trunkfrom
erichdongubler-mozilla:distinguish-diag-filters-and-wgsl

Conversation

@ErichDonGubler
Copy link
Copy Markdown
Member

@ErichDonGubler ErichDonGubler commented Nov 8, 2024

Connections

Resolves #6485.

Description

-

Testing

If this compiles, I consider it tested. 🙂

Checklist

  • Run cargo fmt.
  • Run taplo format.
  • Run cargo clippy. If applicable, add:
    • --target wasm32-unknown-unknown
    • --target wasm32-unknown-emscripten
  • Run cargo xtask test to run tests.
  • Add change to CHANGELOG.md. See simple instructions inside file. No user-facing changes, so none needed.

@ErichDonGubler ErichDonGubler added naga Shader Translator kind: refactor Making existing function faster or nicer lang: WGSL WebGPU Shading Language labels Nov 8, 2024
@ErichDonGubler ErichDonGubler self-assigned this Nov 8, 2024
@ErichDonGubler ErichDonGubler requested a review from a team November 8, 2024 19:00
@ErichDonGubler
Copy link
Copy Markdown
Member Author

Summoning @jimblandy as a reviewer, since he filed the issue motivating this PR (#6485).

@ErichDonGubler ErichDonGubler force-pushed the distinguish-diag-filters-and-wgsl branch from 38a0e55 to a04b1cb Compare November 12, 2024 13:57
@ErichDonGubler
Copy link
Copy Markdown
Member Author

This should be ready for review now, @jimblandy!

@ErichDonGubler
Copy link
Copy Markdown
Member Author

Reviewed in #6533.

@ErichDonGubler ErichDonGubler deleted the distinguish-diag-filters-and-wgsl branch November 13, 2024 13:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind: refactor Making existing function faster or nicer lang: WGSL WebGPU Shading Language naga Shader Translator

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

naga/src/diagnostic_filter.rs is in language-independent code, but defines WGSL syntax

1 participant