-
Notifications
You must be signed in to change notification settings - Fork 130
GCC 2.8.1 Tips and Tricks
This page lists patterns and problems that frequently show up when decomping, and how to go about matching them.
TARGET CURRENT
lui a0, %hi(foo) lui a0, %hi(foo)
addiu a0, %lo(foo)
jal bar jal bar
nop addiu a0, %lo(foo)
This usually means foo
is a .data
or .rodata
symbol of this file, rather than extern
ed elsewhere. (TODO: could it be the other way around? alex has forgotten)
This
f32 x;
MULT_HI(x, constant) >> shift
should be rewritten as
f32 x;
x / y
This issue lists the constants/shifts: https://github.com/NationalSecurityAgency/ghidra/issues/668
e.g. MULT_HI(x, 0x51EB851F) >> 5
is x / 10
You may see the asm do this:
void fx_73_update(EffectInstance* arg0) {
SomeStruct* structTemp;
s32 i;
structTemp = arg0->data;
if (arg0->numParts > 1) {
structPlus20 = temp_a1 + 0x20;
do {
if (structPlus20->unk0 <= 0) {
structPlus20->unk-1C--;
if (structPlus20->unk-1C >= 0xA) {
structPlus20->unk0 = -1;
}
}
i++;
structPlus20 += 0x24;
} while (i < arg0->numParts);
}
}
Note the negative offsets (unk-1C, for example) and how phi_v1 is 0x20 bytes into the struct. You can calculate the correct offsets by taking the 0x20 and subtracting 0x1C to get unk_04. However, to get the code to actually generate these negative offsets, you need to increment the struct temp pointer as well as the normal loop iterator: for (i = 1; i < numParts; i++, part++) {
If you encounter likely instructions, these mean that the delay slot instruction is only executed if the branch is taken. Sometimes your code is equalvent but you still get these instructions (or don't get them when you want them). Try inverting the condition, as this sometimes coerces the compiler into using a likely instruction