Skip to content

monobj: opus2 (mlAttackCheck cache, setRepop)#17669

Open
AsclepiusBot wants to merge 4 commits into
mainfrom
pr/monobj-opus2-1781462565
Open

monobj: opus2 (mlAttackCheck cache, setRepop)#17669
AsclepiusBot wants to merge 4 commits into
mainfrom
pr/monobj-opus2-1781462565

Conversation

@AsclepiusBot

Copy link
Copy Markdown
Collaborator

Recovered opus2 gated wins (agent died pre-push). Each commit strict report.json increase.

🤖 Generated with Claude Code

claude added 4 commits June 14, 2026 11:48
Merging the Pass-2 result accumulator into selectedTarget eliminates a
register-bounce (mr r27,r23) and a separate selectedTarget=-1 init, matching
the target which reuses the same register for result and selectedTarget.
statWatch 96.725->96.905, unit 97.867->97.880.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
The weapon-mode particle count is read from u16 script fields; holding it
in a signed short forced extsh + cast, while the target zero-extends (clrlwi)
and compares directly. int countC drops the cast and matches the loop bound.
setRepop 96.920->97.897, unit 97.880->97.925.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Rewriting the classId dispatch as an explicit (>= 0xA9 -> pbDone) else
(pbSet2) chain emits the target's bge branch instead of our blt, matching
block layout. setRepop 97.897->97.939, unit 97.925->97.927.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
The volatile groupCursor was re-read between the groupCount[] index and the
increment; reading it once into a local wcursor (keeping the volatile store
and the post-store >=8 re-read) matches the target's single-load pattern.
mlAttackCheck 96.493->96.702, unit 97.927->97.940.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@decomp-dev

decomp-dev Bot commented Jun 24, 2026

Copy link
Copy Markdown

Report for GCCP01 (ff63985 - f030f7a)

No changes

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.

2 participants