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

Upgrade Binutils to 2.31.1 #16

Merged
merged 1 commit into from
May 6, 2024
Merged

Upgrade Binutils to 2.31.1 #16

merged 1 commit into from
May 6, 2024

Conversation

fjtrujy
Copy link
Member

@fjtrujy fjtrujy commented May 5, 2024

Description

This PR makes 2 things:

  • Upgrade binutils to 2.31.1, the process to achieve this has been to isolate specific DVP changes from original patches, and then go version by version applying the patches and verify that it is working fine.
  • Add MacOS ARM support.

To check if it was working fine I have been using the @Ziemas repo https://github.com/Ziemas/xtc/tree/ps2sdk

Screenshot 2024-05-06 at 00 37 43

I couldn't upgraded to a newer version, even I have been able to resolved conflicts and make it to works it was producing runtime errors when using it.
Maybe I will try to resume in the future

Cheers.!

@fjtrujy fjtrujy requested review from uyjulian and rickgaiser May 5, 2024 22:39
Copy link
Member

@uyjulian uyjulian left a comment

Choose a reason for hiding this comment

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

Lgtm

@uyjulian uyjulian merged commit eb6b702 into main May 6, 2024
9 checks passed
@fjtrujy fjtrujy deleted the update_2.31.1 branch May 6, 2024 15:35
@F0bes
Copy link

F0bes commented May 6, 2024

It works fine for my PS2DVD example.
There is some issue with the old dvp-as where using a global label as a branch target causes some odd relocation errors. I don't have a reproducible sample, and I'm far too busy to try and make one to see if it's fixed here.

Anyways, it works good. Nice job o7

@Ziemas
Copy link

Ziemas commented May 6, 2024

It works fine for my PS2DVD example. There is some issue with the old dvp-as where using a global label as a branch target causes some odd relocation errors. I don't have a reproducible sample, and I'm far too busy to try and make one to see if it's fixed here.

Anyways, it works good. Nice job o7

The EE toolchain does not have the DVP relocation patches, which probably explains that? IIRC that's something you'd run into with VU0 code though.

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