☀️ 56% – Ready-to-score Effect_Move family and named exact owners - #15
Draft
PascalPixel wants to merge 1 commit into
Draft
PascalPixel wants to merge 1 commit into
PascalPixel wants to merge 1 commit into
Conversation
…ring
This VM has no approved ROMs and cannot install the compiler bundle
(source bootstrap is admitted only on Apple Silicon macOS), so none of
the named owners can be scored or adopted here. DONE stays at the last
verified ☀️ 56%.
Listing comparison of the 54-byte Effect_Move body (Lunpa's already-exact
C) shows thirteen overlays still holding identical instruction bytes
after AlchemyC_020000a0: resource_382, 387, 396, 39b, 39c, 39e, 3a0,
3a5, 3a6, 3b3, 3be, 3c0 and 3c9. They want one shared source with
instances. The five overlay copies of main:0809a65c still compile only
behind a do { } while (0) barrier.
Keep the recovered candidates typed and ready to score on Pascal's
machine; do not register them as exact C without the approved route.
Co-Authored-By: Cursor Grok 4.6 <noreply@cursor.com>
Co-authored-by: Pascal Pixel <x@pascalpixel.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This run could not raise TBS DONE. The VM has no approved ROMs under
roms/and cannot install the compiler bundle (alchemy bootstrapadmits source builds only on Apple Silicon macOS).make progressandmake verifytherefore cannot run here.Before / after
make progress: not runnable on this host. Last verified figure fromorigin/main/ README: ☀️ 55.96% (commit prefix 56%) · ⚓️ 0.20%. This branch does not claim new exact C.Named owners (not adopted)
main:080fa514MusicPlayer_ResetActiveTracksSoundPlayer/SoundTrack. Still assembly (raw/080fa514.s).main:08092878ObjectLink_RotatePairTowardObjectPairPosition. Still assembly (raw/08092878.s).resource_39e:02000104Effect_Moveresource_3c9:02000104Effect_MoveNo owners were adopted. Assets were not touched.
Effect_Move family (listing evidence)
Lunpa (
resource_3ab:02000104) already compilesEffect_Moveas exact C (54 bytes + 2-byte align). The same 27 halfwords sit in the overlay listings immediately afterAlchemyC_020000a0(.space 0x62/.2byte 0x0000) for thirteen overlays, not two:resource_382,387,396,39b,39c,39e,3a0,3a5,3a6,3b3,3be,3c0,3c9That is 13 × 54 = 702 bytes if one shared source with instances matches under the approved route. Four more overlays already have an
AlchemyC_02000104placeholder (380,39d,3a1,3ab) compiling equivalent motion C from other files; fold those in only after sibling scoring.The “eleven more copies differ by one halfword” note is likely 56-byte region records (function + alignment) versus a 54-byte C object, or a different last-store spelling (
record[15]vsobj[7].h[1]). The thirteen listings above are byte-identical to each other.Shared candidate:
games/THE BROKEN SEAL/recon/en/overlays/effect_move.c.0809a65c overlay copies (still a scheduling trick)
Main
0809a65cis ordinary C (ParticleEffect_UpdateMotionAndScale). Five overlay copies still usedo { } while (0)to order the Z load:resource_3b2:02000da4AdvanceEffectMotionresource_3c5:02000cf0advance_effect_motionresource_396:0200185cadvance_effect_motionresource_3c4:02001068advance_effect_motionresource_3c8:0200096cadvance_effect_motionOrdinary candidate (same body as main, bind
__divsi3to each overlay divide veneer):games/THE BROKEN SEAL/recon/en/overlays/advance_effect_motion.c. If it does not match, withdraw; do not keep the barrier.SpawnConfiguredEffect(23 copies) stays a recorded compiler gap.Leftovers
make coverage) and twin ranking need a verified ROM build.Effect_Moveout of LunpaEFFECT.Cinto a shared unit with instances needs scoring of both the remaining Lunpa unit and every instance.Verify / adopt on Apple Silicon (Pascal)
Do not commit ROM bytes. Do not change compiler pins.