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

Update the SPEC file for new-age options #5

Closed
3246251196 opened this issue Nov 23, 2023 · 9 comments
Closed

Update the SPEC file for new-age options #5

3246251196 opened this issue Nov 23, 2023 · 9 comments

Comments

@3246251196
Copy link
Member

@afxgroup
We should discus this on the 26th October. e.g. machine target command line options that are out of date in the SPEC file.

@3246251196
Copy link
Member Author

Bump. For the next meeting after the 7th.

@afxgroup
Copy link

afxgroup commented Jan 8, 2024

Yes, we'll arrange a meeting

@afxgroup
Copy link

Rebump! I need it :P

@3246251196
Copy link
Member Author

@afxgroup
Can you quote what the changes or the bulk of new options are?

@afxgroup
Copy link

If you try to compile something that has specific cpu code it doesn't work anymore with gcc11 (and iirc 8)
For example:
http://www.os4depot.net/?function=showfile&file=utility/shell/sam440ep_setup.lha

But also code with altivec. A good test is also the graphics library

@3246251196
Copy link
Member Author

@afxgroup
Reminder to prove the spec file in this issue when you have the time.

@3246251196
Copy link
Member Author

@afxgroup
https://github.com/AmigaLabs/adtools/tree/gcc11_spec_machine_options
Perhaps we can test this out a little bit on some projects that you know fail. I happened to test it for the sam440 LHA above and it worked.

===
I was wondering why this actually fixed anything since the 440fp option has not changed in this new spec file. For sure, we should update our spec file and I will create the PR, but the real fix is the implicit inclusion of this option:

-many
    Generate code for any architecture (PWR/PWRX/PPC).

due to the last entry for ASM_CPU_SPEC which is ASM_OPT_ANY.
Well, that at least is the fix for the test file that you provided in the ticket. I am sure updating our spec file also fixes other more intricate things!

@3246251196
Copy link
Member Author

@3246251196
TODO: See sba1#164. Merge this into our Master instead.

@3246251196
Copy link
Member Author

Since we agreed to start using our own AmigaLabs fork from now on and this is merged. This issue is now closed and resolved.

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

No branches or pull requests

2 participants