-
Notifications
You must be signed in to change notification settings - Fork 44
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 toolchain and aux libraries #34
base: master
Are you sure you want to change the base?
Conversation
Tested the fat driver, it works. I can't test the ext driver as it is still broken in XeLL. But it does compile. There might well be an issue with it, so if you want me to just remove it from the build script rather than just assume it does lmk. |
can i ask why you closed these? |
XeLL crashes when trying to load an ELF. I need to fix that before it can be merged. |
Are you sure it's XeLL that crashes and not the ELF itself? Also - did you use XeLL (old one) or XeLL Reloaded (new one)? did you use a re-compiled version with your fixes or a existing pre-compiled version? I recommend not closing PRs while investigating issues, just leave them open with a comment explaining that there is a known issue you're working on instead. |
@@ -303,7 +303,7 @@ function filesystems_install | |||
{ | |||
echo -e "Building Filesystems..." | |||
|
|||
filesystems="fat-xenon ext2fs-xenon xtaflib" # ntfs-xenon ext2fs-xenon xtaflib | |||
filesystems="fat-xenon ext2fs-xenon" # xtaflib ntfs-xenon ext2fs-xenon xtaflib |
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.
Why did you disable the xtaflib here?
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.
This was a mistake - I'll need to test if it works with the new toolchain first
(match_operand:V4SI 3 "register_operand" "v")] | ||
UNSPEC_VMSUMU))] | ||
- "TARGET_ALTIVEC" | ||
+ "(TARGET_ALTIVEC && 0)" |
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.
Why disable all of these? && 0 is equivalent to false?
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.
Yes, this is disabling the Altivec instructions which do not exist on the Xenon CPU. All of the GCC patches do this.
I recompiled XeLL Reloaded from the master branch with the new toolchain. I'm sure it's XeLL that crashes, as existing ELFs which work fine on older versions are crashing |
DON'T MERGE YET. LINUX LOADING/ELF LOADING IS BROKEN. I NEED TO MAKE CHANGES TO LIBXENON.
I also fixed up the fat-xenon and xenon-ext2fs filesystem drivers (yet to be tested but code changes I made were relatively minor and they compile), and I will look at fixing the others soon-ish. So please have a look at those pull requests as well, as the build script fails on the current versions on here.