Skip to content

Commit d5754d0

Browse files
committed
Add tracing to rustc_codegen_spirv
Fixes #192
1 parent 854e9ba commit d5754d0

File tree

6 files changed

+271
-52
lines changed

6 files changed

+271
-52
lines changed

.github/workflows/lint.sh

+8-2
Original file line numberDiff line numberDiff line change
@@ -49,10 +49,16 @@ clippy_no_features examples/shaders/simplest-shader
4949
# which could be disastrous because env vars access can't be tracked by
5050
# `rustc`, unlike its CLI flags (which are integrated with incremental).
5151
if (
52-
egrep -r '::\s*env|env\s*::' crates/rustc_codegen_spirv/src/ |
52+
egrep -r '::\s*env|env\s*::' crates/rustc_codegen_spirv/src |
5353
# HACK(eddyb) exclude the one place in `rustc_codegen_spirv`
5454
# needing access to an env var (only for codegen args `--help`).
55-
egrep -v '^crates/rustc_codegen_spirv/src/codegen_cx/mod.rs: let help_flag_comes_from_spirv_builder_env_var = std::env::var\(spirv_builder_env_var\)$'
55+
egrep -v '^crates/rustc_codegen_spirv/src/codegen_cx/mod.rs: let help_flag_comes_from_spirv_builder_env_var = std::env::var\(spirv_builder_env_var\)$' |
56+
# HACK(LegNeato) exclude logging. This mirrors `rustc` (`RUSTC_LOG`) and
57+
#`rustdoc` (`RUSTDOC_LOG`).
58+
# There is not a risk of this being disastrous as it does not change the build settings.
59+
egrep -v '^crates/rustc_codegen_spirv/src/lib.rs:.*(RUSTGPU_LOG|RUSTGPU_LOG_FORMAT|RUSTGPU_LOG_COLOR).*$' |
60+
egrep -v '^crates/rustc_codegen_spirv/src/lib.rs: use std::env::{self, VarError};$'
61+
5662
); then
5763
echo '^^^'
5864
echo '!!! Found disallowed `std::env` usage in `rustc_codegen_spirv` !!!'

0 commit comments

Comments
 (0)