Skip to content

Commit

Permalink
No need to cache the profiler_runtime flag
Browse files Browse the repository at this point in the history
This cache struct entry was a relic from when profiler availability was
communicated via an environment variable rather than a command-line flag.
  • Loading branch information
Zalathar authored and liwagu committed Oct 10, 2024
1 parent c07945c commit f8db3d6
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions src/tools/compiletest/src/header/needs.rs
Original file line number Diff line number Diff line change
Expand Up @@ -101,7 +101,7 @@ pub(super) fn handle_needs(
},
Need {
name: "needs-profiler-runtime",
condition: cache.profiler_runtime,
condition: config.profiler_runtime,
ignore_reason: "ignored when the profiler runtime is not available",
},
Need {
Expand Down Expand Up @@ -220,7 +220,6 @@ pub(super) struct CachedNeedsConditions {
sanitizer_memtag: bool,
sanitizer_shadow_call_stack: bool,
sanitizer_safestack: bool,
profiler_runtime: bool,
xray: bool,
rust_lld: bool,
dlltool: bool,
Expand All @@ -247,7 +246,6 @@ impl CachedNeedsConditions {
sanitizer_memtag: sanitizers.contains(&Sanitizer::Memtag),
sanitizer_shadow_call_stack: sanitizers.contains(&Sanitizer::ShadowCallStack),
sanitizer_safestack: sanitizers.contains(&Sanitizer::Safestack),
profiler_runtime: config.profiler_runtime,
xray: config.target_cfg().xray,

// For tests using the `needs-rust-lld` directive (e.g. for `-Clink-self-contained=+linker`),
Expand Down

0 comments on commit f8db3d6

Please sign in to comment.