[TheAggie]

What

ReadRawComputeShader (the fallback for compute programs that were never registered) reads a growing prefix of guest memory, 64, 128, 256 words and so on, and decodes it. When the decoder throws std::out_of_range, it reads more. The instruction-length check, the scalar decoder and the vector decoders throw std::out_of_range for an instruction cut off by the end of the span.

The memory decoders (SMEM, DS, MUBUF, MTBUF, FLAT, through requireTwoWords and the generic memory check), the image decoder (MIMG and its NSA payload) and the export decoder threw std::runtime_error instead. So when a two-word instruction of those families lay across a window boundary, the read failed instead of growing. Whether it happened depended on where the program's instructions fell. vkQuake (PPSA99010) terminated with truncated FLAT instruction in some runs and not others.

Those five throw sites now use std::out_of_range, like the rest of the decoder. The messages are unchanged. Nothing in the tree catches std::runtime_error around decoding; the one test that matches a truncation message (LdsCondxchg, truncated DS instruction) catches std::exception.

Test (agc_driver): a raw program of 63 s_nop, a two-word s_load_dword in words 63 and 64, and s_endpgm reads back as 66 words. It fails (truncated SMEM instruction) without the change.

Tested

Windows 11 Pro (26200), MinGW-w64 GCC 15.2.0 (WinLibs r7), Release, NVIDIA GeForce RTX 5070 Ti.

• The full suite (488 tests) passes on this branch alone. It was run from the generated CTestTestfile.cmake commands because antivirus blocks ctest.exe on this machine.
• tools/check_conventions.py --base origin/main passes.

Not tested on Linux or macOS.

Checklist

• Based on current main; no other open PR implements the same functions
• One topic per PR; follow-ups go in a new PR
• No comments in code except technical debt
• Unimplemented paths throw (NotImplemented_nid_no_patch); silent stubs are listed in TechnicalDebt
• No notes, investigation .md files or images added to the repository (attach them to this PR)
• New third-party code is a submodule built from source
• Generic behaviour, not specific to one title
• Depends on: none
• AI-assisted: yes

🤖 Generated with Claude Code

[MarcZX10]

Reviewed the complete current diff. Only truncated-span checks in the memory, image/NSA, and export decoders change exception type; the diagnostic strings and invalid-PC checks remain unchanged. The regression crosses the initial 64-word read window, and current Linux, Windows, conventions, and report checks are green. Approved.

[adriadam10]

Checked: build + ctest (libSceAgc, libSceAgcDriver) green locally; agc_driver fails with the three decoder files reverted and passes with them. Every remaining "truncated" throw in the decoders is now std::out_of_range, so ReadRawComputeShader grows its window for all of them. Merging.