EEGCC output's bad asm / asm alignment. about splat HOT 5 CLOSED

This issue does not occur on the normal GCC compiler.

from splat.

thanks for the report @weirdbeardgame - can you link a repro branch?

from splat.

https://github.com/weirdbeardgame/Fatal-Frame-2 - Uncomment Gphase.c and the file right below it then try and run it

from splat.

This behavior is expected due to how the INCLUDE_ASM macro is defined.
What is happening is both your INCLUDE_ASM macro and the function asm file are defining the same symbol (InitGPhaseSys__Fv in this case) by the "\t.globl\t" #NAME "\n" and the glabel InitGPhaseSys__Fv respectively.

The processed asm ends up looking something like this:

.section .text
    .align    3
    .globl    InitGPhaseSys__Fv
    .ent    InitGPhaseSys__Fv
InitGPhaseSys__Fv:
    .set noreorder
    .set noat
.align 3
glabel InitGPhaseSys__Fv
/* B09A8 001AF9A8 F0FFBD27 */  addiu      $29, $29, -0x10
# ...

Notice how the symbol is defined twice. This usually doesn't matter, because in normal circumstances there's nothing that may change the alignment of the symbol between the first and second declaration, but here there's an alignment directive that (in the eyes of the assembler) may change the address of the symbol, then the assembler decides to bark at you.

You can rewrite the INCLUDE_ASM macro to something like this to fix it:

#define INCLUDE_ASM(FOLDER, NAME) \
    __asm__( \
        ".section .text\n" \
        "    .set noat\n" \
        "    .set noreorder\n" \
        "    .include \""FOLDER"/"#NAME".s\"\n" \
        "    .set reorder\n" \
        "    .set at\n" \
        "    .globl    " #NAME ".NON_MATCHING\n" \
        "    " #NAME ".NON_MATCHING" " = " #NAME "\n" \
    )

The important part is the macro is not defining the symbol again, it is letting the asm file to do that instead.
I know it works because that macro is what I'm using at hit_and_run

btw, why is your INCLUDE_ASM macro so cursed? why do you even need something like INCLUDE_ASM_INTERNAL?

from splat.

That seemed to fix this issue. Thanks!

from splat.