Skip to content
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.

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

Small changes from the OpenGL branch #321

Merged
merged 8 commits into from
Dec 21, 2022
Merged

Conversation

rasky
Copy link
Collaborator

@rasky rasky commented Dec 9, 2022

This is another set of unrelated changes coming from the OpenGL. These can be merged safely even right now.

@rasky rasky requested a review from anacierdem December 9, 2022 23:26
@rasky rasky force-pushed the rdpq_merge branch 2 times, most recently from 0f336e1 to dd6d137 Compare December 10, 2022 06:48
Copy link
Collaborator

@anacierdem anacierdem left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! There is only a single typo on a macro name that I'm not sure if its correct and we can add the linker script as an elf dependency. o/w nothing to worry about.

include/pputils.h Show resolved Hide resolved
include/pputils.h Outdated Show resolved Hide resolved
include/rsp_queue.inc Outdated Show resolved Hide resolved
n64.mk Show resolved Hide resolved
rsp.ld Show resolved Hide resolved
Makefile Show resolved Hide resolved
tests/testrom.c Show resolved Hide resolved
@rasky rasky force-pushed the rdpq_merge branch 2 times, most recently from 74ac458 to 91f6aac Compare December 18, 2022 15:14
This allows to define in the future multiple text/data sections and
rely on the linker to remove those that are not used. It is useful
in the context of RSP reusable code libraries.
While writing vectorized RSP code, it is quite useful to have a zero
register (initialized to all zero) and also various power of twos to
implement shifts.

This commit changes rspq so that any time it calls a function in an
overlay, it first initializes $v00 to zero, and $v30/$v30 with the
constants required to implement the shift macros (see rsp.inc).
a different name so to be able to combine both if required):
…ail.

This allows to commit log messages in tests, as they don't pollute the
debugging spew. The text will be shown if the test ever fails.

Also introduce a ERR() macro for text that should be displayed on both
stdout and stderr, again only when the test fails.
We shouldn't apply the scanline shifting logic in non-interlaced modes.
@rasky rasky merged commit 038e388 into DragonMinded:trunk Dec 21, 2022
@rasky rasky deleted the rdpq_merge branch December 21, 2022 13:03
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.

4 participants