Skip to content

Add XGCompiler class - #13

Open
trevor403 wants to merge 2 commits into
masterfrom
rad.compiler-class
Open

Add XGCompiler class#13
trevor403 wants to merge 2 commits into
masterfrom
rad.compiler-class

Conversation

@trevor403

Copy link
Copy Markdown
Collaborator

Based on devkitppc

}
#endif

class func replaceRamASM(RAMOffset: Int, newASM asm: [UInt32]) {

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Do you want to keep this? I think I removed it and switched to the metroworks area in dol as the preference. This could still be useful in future if we end up writing a lot of assembly code. It's region specific though which is the main reason I didn't like it as a solution.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

This is actually just the UInt32 version of the function, they are identical with different return types

enum Commands {
case wit
case wimgt
case gcc

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

The indentation is a bit funny here but no big deal

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

whoops

return output
}

@discardableResult

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Did you mean to remove this?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Nope, my bad!

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.

2 participants