-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
PowerPC: Thread state through float helpers #7889
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You may want to consider passing by reference in cases where relevant state cannot be null, to enforce that at a type system level. Particularly given the changes are transitional.
Source/Core/Core/PowerPC/Interpreter/Interpreter_FloatingPoint.cpp
Outdated
Show resolved
Hide resolved
Source/Core/Core/PowerPC/Interpreter/Interpreter_SystemRegisters.cpp
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@CrystalGamma Please use the PowerPC prefix for relevant commit messages and resolve the comments you've addressed.
I wanted to squash at the end when the code changes are approved, since those commits are only changes to the changes in the original PR commit. |
Re: pointers/references: the code style guidelines say "Functions that specifically modify their parameters should have the respective parameter(s) marked as a pointer so that the variables being modified are syntaxically obvious" so that's why I did that. I'm open to changing that though should you insist … |
c960e35
to
23168ba
Compare
23168ba
to
50b1470
Compare
50b1470
to
b00a704
Compare
No description provided.