Skip to content

ido7.1_irix4 added - #2068

Open
jtl3d wants to merge 2 commits into
decompme:mainfrom
jtl3d:ido7.1_irix4
Open

ido7.1_irix4 added#2068
jtl3d wants to merge 2 commits into
decompme:mainfrom
jtl3d:ido7.1_irix4

Conversation

@jtl3d

@jtl3d jtl3d commented Aug 8, 2026

Copy link
Copy Markdown

This is an attempt to get a more generic solution added than what was attempted in #2060. Since all the binaries are there, it uses a script to create the compiler. Hopefully this is acceptable, but if not please suggest alternatives!

@mkst

mkst commented Aug 8, 2026

Copy link
Copy Markdown
Collaborator

Any script to run the compiler should be packaged along with the compiler, not part of this repo

@jtl3d

jtl3d commented Aug 8, 2026

Copy link
Copy Markdown
Author

Which compiler? The ido 7.1 one? Or a new one?

@mkst

mkst commented Aug 8, 2026

Copy link
Copy Markdown
Collaborator

Any compiler. But that cc-irix script does not belong here 😊

@jtl3d

jtl3d commented Aug 8, 2026

Copy link
Copy Markdown
Author

Yeah makes sense... will move to compilers. Thanks!

@jtl3d

jtl3d commented Aug 8, 2026

Copy link
Copy Markdown
Author

@mkst OK I created decompme/compilers#79 to have the script live there. Let me know if anything else needs changing!

platform=N64,
base_compiler=IDO71,
cc='"${COMPILER_DIR}/cc-irix4"'
' --frontend "${COMPILER_DIR}/../ido4.1" --backend "${COMPILER_DIR}"'

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.

Have you tested this? I was (am?) expecting the ido 7.1 bundle to include irix4 along with qemu-irix and whatever else is required, maybe ../ works but it would need testing to confirm..

@mkst

mkst commented Aug 8, 2026

Copy link
Copy Markdown
Collaborator

Sorry for my terseness, I'm not home and replying on mobile which isn't a particularly enjoyable experience 😅

@jtl3d

jtl3d commented Aug 8, 2026

Copy link
Copy Markdown
Author

No rush to test! I had to run and I'm not sure if i messed something up in my haste so I'll look deeper when i get another chance

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