early linting: avoid redundant calls to check_id

[fee1-dead]

An attempt to address the regression at #142240 (comment)

r? @oli-obk

cc @nnethercote who might have a better understanding of the performance implications

[fee1-dead]

@bors try @rust-timer queue

[bors]

⌛ Trying commit ac92e87 with merge 73e31b9...

[bors]

💔 Test failed - checks-actions

[oli-obk]

Sad. Would need to revert #142305, too then. At this point it may be prudent to remove visit_id entirely as it's a footgun

[fee1-dead]

it might still be possible to keep using visit_id, I just haven't looked into it deeply yet. After removing the redundant calls we can do some perf measurements. If it still is bad then we should revert to previous state.

[fee1-dead]

@bors try @rust-timer queue

[bors]

⌛ Trying commit ae8ca1f with merge 9fe6114...

[bors]

☀️ Try build successful - checks-actions
Build commit: 9fe6114 (9fe61147dc9031465e04140f03586e3d72f36e6c)

[rust-timer]

Finished benchmarking commit (9fe6114): comparison URL.

Overall result: ✅ improvements - no action needed

Benchmarking this pull request means it may be perf-sensitive – we'll automatically label it not fit for rolling up. You can override this, but we strongly advise not to, due to possible changes in compiler perf.

@bors rollup=never
@rustbot label: -S-waiting-on-perf -perf-regression

Instruction count

Our most reliable metric. Used to determine the overall result above. However, even this metric can be noisy.

	mean	range	count
Regressions ❌ (primary)	-	-	0
Regressions ❌ (secondary)	-	-	0
Improvements ✅ (primary)	-0.3%	[-0.6%, -0.1%]	30
Improvements ✅ (secondary)	-0.4%	[-1.0%, -0.0%]	36
All ❌✅ (primary)	-0.3%	[-0.6%, -0.1%]	30

Max RSS (memory usage)

Results (primary 0.4%)

A less reliable metric. May be of interest, but not used to determine the overall result above.

	mean	range	count
Regressions ❌ (primary)	1.3%	[1.2%, 1.4%]	2
Regressions ❌ (secondary)	-	-	0
Improvements ✅ (primary)	-1.5%	[-1.5%, -1.5%]	1
Improvements ✅ (secondary)	-	-	0
All ❌✅ (primary)	0.4%	[-1.5%, 1.4%]	3

Cycles

Results (secondary -7.0%)

A less reliable metric. May be of interest, but not used to determine the overall result above.

	mean	range	count
Regressions ❌ (primary)	-	-	0
Regressions ❌ (secondary)	-	-	0
Improvements ✅ (primary)	-	-	0
Improvements ✅ (secondary)	-7.0%	[-7.0%, -7.0%]	1
All ❌✅ (primary)	-	-	0

Binary size

This benchmark run did not return any relevant results for this metric.

Bootstrap: 755.158s -> 754.58s (-0.08%)
Artifact size: 372.13 MiB -> 372.08 MiB (-0.01%)

[oli-obk]

@bors r+ yay

[bors]

📌 Commit ae8ca1f has been approved by oli-obk

It is now in the queue for this repository.