Skip to content

Commit 7a0e8bd

Browse files
committed
No need to cache the profiler_runtime flag
This cache struct entry was a relic from when profiler availability was communicated via an environment variable rather than a command-line flag.
1 parent 8320a01 commit 7a0e8bd

File tree

1 file changed

+1
-3
lines changed
  • src/tools/compiletest/src/header

1 file changed

+1
-3
lines changed

Diff for: src/tools/compiletest/src/header/needs.rs

+1-3
Original file line numberDiff line numberDiff line change
@@ -101,7 +101,7 @@ pub(super) fn handle_needs(
101101
},
102102
Need {
103103
name: "needs-profiler-runtime",
104-
condition: cache.profiler_runtime,
104+
condition: config.profiler_runtime,
105105
ignore_reason: "ignored when the profiler runtime is not available",
106106
},
107107
Need {
@@ -220,7 +220,6 @@ pub(super) struct CachedNeedsConditions {
220220
sanitizer_memtag: bool,
221221
sanitizer_shadow_call_stack: bool,
222222
sanitizer_safestack: bool,
223-
profiler_runtime: bool,
224223
xray: bool,
225224
rust_lld: bool,
226225
dlltool: bool,
@@ -247,7 +246,6 @@ impl CachedNeedsConditions {
247246
sanitizer_memtag: sanitizers.contains(&Sanitizer::Memtag),
248247
sanitizer_shadow_call_stack: sanitizers.contains(&Sanitizer::ShadowCallStack),
249248
sanitizer_safestack: sanitizers.contains(&Sanitizer::Safestack),
250-
profiler_runtime: config.profiler_runtime,
251249
xray: config.target_cfg().xray,
252250

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

0 commit comments

Comments
 (0)