fix(dsp): drop INLINE from DSPExec to fix MSVC link error

DSPExec is declared in dsp.h with external linkage but was defined
INLINE (which expands to MSVC's _inline) in dsp.c. At v2.2.0 the
function was small enough that MSVC always inlined it, so no external
symbol was needed. Recent DSP changes (computed-goto, inline switch
dispatch, delay-slot inlining) inflated the function past MSVC's
inline budget — MSVC then emits the body with internal linkage only,
which causes 'unresolved external symbol _DSPExec' in jaguar.c at
Windows link time.

GPUExec (the parallel function in src/tom/gpu.c) is correctly defined
without INLINE for the same reason. Match that.

Also update RELEASE_NOTES_v2.3.0.md commit/file/line counts to match
the current branch tip (158 commits / 263 files).

Co-Authored-By: Claude Opus 4.7 <[email protected]>
This commit is contained in:
Joseph MattielloandClaude Opus 4.7 committed 2026-05-06 20:17:15 -04:00
1 parent c4e3d2dd2d
commit 2ef50e4a4a
2 files changed
+3 -3

No files matched your search

+2 -2
View File
@@ -1,6 +1,6 @@
# Virtual Jaguar libretro v2.3.0
Performance and accuracy release — 154 commits since v2.2.0. Major blitter
Performance and accuracy release — 158 commits since v2.2.0. Major blitter
and RISC performance work, hardware accuracy fixes across TOM/JERRY/DSP, and
improved iOS/Provenance stability. No new features; this is a polish release.
@@ -86,7 +86,7 @@ improved iOS/Provenance stability. No new features; this is a polish release.
## Compared to v2.2.0
261 files changed, +25,096 / -2,800 lines across 154 commits.
263 files changed, +25,240 / -2,805 lines across 158 commits.
## Downloads
+1 -1
View File
@@ -836,7 +836,7 @@ void DSPDone(void)
/* DSP execution core */
INLINE void DSPExec(int32_t cycles)
void DSPExec(int32_t cycles)
{
#ifdef DSP_SINGLE_STEPPING
if (dsp_control & 0x18)