Skip to content

Add a warning for functions with no size #62

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
AngheloAlf wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Add a warning for functions with no size #62

AngheloAlf wants to merge 3 commits into from

Conversation

@AngheloAlf
Copy link
Contributor

@AngheloAlf AngheloAlf commented May 21, 2024

It can be useful to figure out problems earlier instead of having to deal with cryptic errors from the linker.

It prints a message like this.

[WARN] Function 'func_8007C6D8' has zero size.

Sadly it may catch some RSP microcodes depending on how the repo is setup

[WARN] Function 'gspF3DEX2_fifoTextEnd' has zero size.
Found entrypoint, original name: entrypoint
[WARN] Function 'aspMainTextStart' has zero size.
[WARN] Function 'aspMainTextEnd' has zero size.
[WARN] Function 'func_8007C6D8' has zero size.
[WARN] Function 'gspS2DEX_fifoTextEnd' has zero size.
[WARN] Function 'rspbootTextEnd' has zero size.
[WARN] Function 'gspS2DEX_fifoTextStart' has zero size.

A possible workaround is to add all known microcodes to the ignored_funcs list. I didn't include that in this PR because I didn't knew if you would like the idea, but I have no problem doing it if you like it.

@AngheloAlf AngheloAlf force-pushed the warn_zero_size_functions branch from efc9018 to dd71e92 Compare July 31, 2024 19:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Mr-Wiseguy Mr-Wiseguy Mr-Wiseguy approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@AngheloAlf @Mr-Wiseguy