Skip to content

Reconstruct five object profile pairs - #2140

Merged
andrewboudreau merged 2 commits into
mainfrom
cpp/profile-reconstruction-wave10
Sep 2, 2026
Merged

Reconstruct five object profile pairs#2140
andrewboudreau merged 2 commits into
mainfrom
cpp/profile-reconstruction-wave10

Conversation

@andrewboudreau

Copy link
Copy Markdown
Collaborator

Stacked on #2139. Reconstructs the registry-facing factory/profile pairs for PILE, BK_BILLBOARD, BK_KILLER_DAI, BK_BOTAOSI, and BK_DOWN_B: daObjPile_c_classInit / g_profile_PILE; daObjBkBillboard_c_classInit / g_profile_BK_BILLBOARD; daObjBkKillerdai_c_classInit / g_profile_BK_KILLER_DAI; daObjBk_Botaosi_c_classInit / g_profile_BK_BOTAOSI; daObjBk_Fall_Block_c_classInit / g_profile_BK_DOWN_B. These preserve the separation between ROM RTTI class identity and existing community implementation aliases such as Stump, PoleBillboard, BillBlaster, KnockDownPlank, and FallBlockWf. Exact original SM64DS spellings are not claimed, and no source-owned profile descriptor data is invented. Validation: all five factories match 2004/b56 with strict relocation checks; all 39 affected sources are VERIFIED; five history-preserving path moves keep attribution intact; 98/98 reconstructed TUs compile; 11,088/11,088 source functions and 4/4 source data claims reproduce; 106/106 modules exact; repository reference, attribution, compatibility, language-mode, and dead-reference gates pass.

@tangos-validator

tangos-validator Bot commented Sep 1, 2026

Copy link
Copy Markdown

✅ PR validation — Passed

Committed merge introduces no reconstruction or attribution regression.

Full merge validation

Check Result
Committed test merge yes
Byte-verified functions 10,765 / 11,347 (94.87%, +0)
Byte-verified code bytes 2,002,192 / 2,211,124 (90.55%, +0)
Claimed, not byte-verified 452 functions, 103,488 bytes (+0)
Perfect source moves 0 R100
Enrolled ranges (delinks complete) 11,059 functions, 2,053,148 bytes (92.86%, +0) -- differs from byte-verified by +294
Contributor credit 0 added, 0 changed, 0 lost
Relocation check 39 checked; 39 VERIFIED
Port reference check 405 checked; 0 stale
Module fidelity 106/106 exact; 100.000000% compared bytes
Code linked from verified source 11,088 functions, 2,067,148 bytes (93.49%)
Module bytes from source 2,067,148 / 3,049,600 (67.8%); 811,492 (26.6%) are data no delink entry reaches
ROM data reproduced from source 564 symbol(s) exact, 161 partial, 5 differ

Byte-verified means the range carries complete in a delinks.txt, so the ROM build compiled it and compared it to the cartridge. The 452 claimed functions have a src/ file named after the symbol with no NONMATCHING banner, and nothing compiles them -- dsd fills their addresses with the ROM's own bytes. Both together are the 11,217 this project calls matched.

Per-file link-check detail

All 39 changed file(s) compile to the ROM byte-for-byte with correct relocation targets.

File Symbol Result Slots checked
src/_ZN11BillBlaster13InitResourcesEv.cpp _ZN11BillBlaster13InitResourcesEv ✅ verified 1
src/_ZN11BillBlaster15OnHitByMegaCharER6Player.cpp _ZN11BillBlaster15OnHitByMegaCharER6Player ✅ verified 1
src/_ZN11BillBlaster16CleanupResourcesEv.cpp _ZN11BillBlaster16CleanupResourcesEv ✅ verified 1
src/_ZN11BillBlaster4KillEv.cpp _ZN11BillBlaster4KillEv ✅ verified 1
src/_ZN11BillBlaster6RenderEv.cpp _ZN11BillBlaster6RenderEv ✅ verified 1
src/_ZN11BillBlaster8BehaviorEv.cpp _ZN11BillBlaster8BehaviorEv ✅ verified 1
src/_ZN11BillBlasterD0Ev.cpp _ZN11BillBlasterD0Ev ✅ verified 1
src/_ZN11BillBlasterD1Ev.cpp _ZN11BillBlasterD1Ev ✅ verified 1
src/_ZN11FallBlockWf13InitResourcesEv.cpp _ZN11FallBlockWf13InitResourcesEv ✅ verified 1
src/_ZN11FallBlockWf16CleanupResourcesEv.cpp _ZN11FallBlockWf16CleanupResourcesEv ✅ verified 1
src/_ZN11FallBlockWfD0Ev.cpp _ZN11FallBlockWfD0Ev ✅ verified 1
src/_ZN11FallBlockWfD1Ev.cpp _ZN11FallBlockWfD1Ev ✅ verified 1
src/_ZN13PoleBillboard13InitResourcesEv.cpp _ZN13PoleBillboard13InitResourcesEv ✅ verified 1
src/_ZN13PoleBillboard16CleanupResourcesEv.cpp _ZN13PoleBillboard16CleanupResourcesEv ✅ verified 1
src/_ZN13PoleBillboard6RenderEv.cpp _ZN13PoleBillboard6RenderEv ✅ verified 1
src/_ZN13PoleBillboardD0Ev.cpp _ZN13PoleBillboardD0Ev ✅ verified 1
src/_ZN13PoleBillboardD1Ev.cpp _ZN13PoleBillboardD1Ev ✅ verified 1
src/_ZN14KnockDownPlank11OnAttacked2ER8dActor_c.cpp _ZN14KnockDownPlank11OnAttacked2ER8dActor_c ✅ verified 1
src/_ZN14KnockDownPlank13InitResourcesEv.cpp _ZN14KnockDownPlank13InitResourcesEv ✅ verified 1
src/_ZN14KnockDownPlank15OnHitByMegaCharER6Player.cpp _ZN14KnockDownPlank15OnHitByMegaCharER6Player ✅ verified 1
src/_ZN14KnockDownPlank16CleanupResourcesEv.cpp _ZN14KnockDownPlank16CleanupResourcesEv ✅ verified 1
src/_ZN14KnockDownPlank6RenderEv.cpp _ZN14KnockDownPlank6RenderEv ✅ verified 1
src/_ZN14KnockDownPlank8BehaviorEv.cpp _ZN14KnockDownPlank8BehaviorEv ✅ verified 1
src/_ZN14KnockDownPlank8OnKickedER8dActor_c.cpp _ZN14KnockDownPlank8OnKickedER8dActor_c ✅ verified 1
src/_ZN14KnockDownPlankD0Ev.cpp _ZN14KnockDownPlankD0Ev ✅ verified 1
src/_ZN14KnockDownPlankD1Ev.cpp _ZN14KnockDownPlankD1Ev ✅ verified 1
src/_ZN5Stump13InitResourcesEv.cpp _ZN5Stump13InitResourcesEv ✅ verified 1
src/_ZN5Stump15OnGroundPoundedER8dActor_c.cpp _ZN5Stump15OnGroundPoundedER8dActor_c ✅ verified 1
src/_ZN5Stump15OnHitByMegaCharER6Player.cpp _ZN5Stump15OnHitByMegaCharER6Player ✅ verified 1
src/_ZN5Stump16CleanupResourcesEv.cpp _ZN5Stump16CleanupResourcesEv ✅ verified 1
src/_ZN5Stump6RenderEv.cpp _ZN5Stump6RenderEv ✅ verified 1
src/_ZN5Stump8BehaviorEv.cpp _ZN5Stump8BehaviorEv ✅ verified 1
src/_ZN5StumpD0Ev.cpp _ZN5StumpD0Ev ✅ verified 1
src/_ZN5StumpD1Ev.cpp _ZN5StumpD1Ev ✅ verified 1
src/daObjBkBillboard_c_classInit.c daObjBkBillboard_c_classInit ✅ verified 1
src/daObjBkKillerdai_c_classInit.c daObjBkKillerdai_c_classInit ✅ verified 1
src/daObjBk_Botaosi_c_classInit.c daObjBk_Botaosi_c_classInit ✅ verified 1
src/daObjBk_Fall_Block_c_classInit.c daObjBk_Fall_Block_c_classInit ✅ verified 1
src/daObjPile_c_classInit.c daObjPile_c_classInit ✅ verified 1

The private worker commits a test merge, builds the stock ROM profile, compares every executable module, measures matched and source-built code, checks contributor lineage, and verifies affected relocations. The mod profile is opt-in and is not part of this merge gate.

@andrewboudreau
andrewboudreau marked this pull request as ready for review September 2, 2026 04:53
Base automatically changed from cpp/profile-reconstruction-wave9 to main September 2, 2026 06:11
@andrewboudreau
andrewboudreau merged commit 3fff345 into main Sep 2, 2026
9 checks passed
@andrewboudreau
andrewboudreau deleted the cpp/profile-reconstruction-wave10 branch September 2, 2026 06:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant