-
-
Notifications
You must be signed in to change notification settings - Fork 81
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
Assembly failure on armv7 #11
Comments
That's in an AngelScript ASM file. I wonder if it's an AngelScript ARMv7 issue? |
Looking at as_callfunc_arm_gcc.S, it appears that this issue is due to it trying to compile the ASM file for the Thumb ISA when it should use the regular ARM ISA. I'll have to see whether I can replicate this issue and maybe fix it in the Makefile. |
Reproduced under Debian Buster armhf with GCC 8.3.0.
|
I managed to make the AngelScript runtime build on armv7l architectures by adding This modifies only the Theoretically this modification should not impact building NymphCast on other platforms, but it should be tested regardless. |
Doesn't seem to work for me, it still fails on armv7. https://gitlab.alpinelinux.org/PureTryOut/aports/-/jobs/90488 |
The Makefile checks with |
Ah makes sense, but that isn't a reliable check. In Alpine's case, armhf and armv7 packages are actually built on a 32-bit capable aarch64 system. |
I have removed the use of uname. Instead the Makefile uses the |
Still no luck sadly, not sure what's up... https://gitlab.alpinelinux.org/PureTryOut/aports/-/jobs/90618 |
I'm not sure what kind of GCC toolchain is used there. It should target arm-linux-gnu, no? The Makefile just checks for a string that starts with |
Just the usual. On my local (x86_64) machine
I'm guessing that should be |
It should start with |
I have changed the Makefile to output the GCC triplet, so we can more easily see what |
https://gitlab.alpinelinux.org/PureTryOut/aports/-/jobs/91563 |
Huh, that's an odd one. Only seen similar triplets with clang so far. I have added the |
Yup that worked, thanks! 😄 |
On armv7, we get an assembly failure on Alpine Linux. See the CI pipeline here.
The error:
The text was updated successfully, but these errors were encountered: