spu: remove outdated bounds check #19908

Merged
Megamouse opened 3:07pm on October 21, 2026 wanted to merge 71 Ξ” into rpcs3/rpcs3 master from
spu_stuff_2

Pull Request Overview

  • Opened on October 21, 2026
  • Status Merged
  • Commit count 2 with first commit October 21, 2026

Total Delta

71 Total Diff Delta

Open Days

Open 17 weekdays

Test Delta

0 Diff Delta in Test Files
Breakdown by Phase

How long has this pull request spent in each phase of its lifecycle?

Fraction of total time Business days Phase
 
0.2 days Authoring 1 commit before pull request opened for review
 
0.4 days Awaiting first review
 
0.1 days Revising work with 1 commit in response to 2 reviews that left 3 comments
Set up deploy tracking to begin tracking full PR start-to-deploy times

Total time for pull request 0.7 business days from first commit authored to merge
Deploy tracking has not been set up

Author avatar

spu: remove outdated bounds check


  • ~Only create and check spu hash if bounds are set~

  • Remove outdated bounds check

We currently unconditionally calculate a hash for spu programs even when we don't set bounds (which is the default).
The hashes are only used to skip functions with these bounds checks.
Apparently no one uses it anyway, so let's just remove them.

Resolved Comment Threads

elad335 reviewed on October 21, 2026
elad335 reviewed on October 21, 2026
rpcs3/Emu/Cell/SPUCommonRecompiler.cpp
Outdated
1037
		// Ensure some actions are performed on a single thread
1038
		const bool is_first_thread = func_i == 0;
1039
1040
		// Get bounds
1041
		const u64 spu_llvm_lower_bound = g_cfg.core.spu_llvm_lower_bound;