8000 ports/unix: Apple M1 on macOS 13.4 error during compilation of py/emitglue.c with MICROPY_FORCE_32BIT=1 · Issue #11760 · micropython/micropython · GitHub
[go: up one dir, main page]
Skip to content

ports/unix: Apple M1 on macOS 13.4 error during compilation of py/emitglue.c with MICROPY_FORCE_32BIT=1 #11760

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.

Sign up for GitHub

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

Jump to bottom
Closed
ned-pcs opened this issue Jun 11, 2023 · 8 comments
Closed

ports/unix: Apple M1 on macOS 13.4 error during compilation of py/emitglue.c with MICROPY_FORCE_32BIT=1 #11760

ned-pcs opened this issue Jun 11, 2023 · 8 comments
Labels
bug

Comments

@ned-pcs
Copy link
Contributor
ned-pcs commented Jun 11, 2023

After reporting #11754 I tried to build using MICROPY_FORCE_32BIT=1, and got this error during compilation of py/emitglue.c:

CC ../../py/emitglue.c
clang -m32 -DMICROPY_VFS_FAT=1 -DMICROPY_VFS_LFS1=1 -DMICROPY_VFS_LFS2=1 -DMICROPY_PY_SSL=1 -DMBEDTLS_CONFIG_FILE=\"mbedtls/mbedtls_config.h\" -DMICROPY_SSL_MBEDTLS=1 -I../../lib/mbedtls/include -DMICROPY_PY_BTREE=1 -DFFCONF_H=\"lib/oofatfs/ffconf.h\" -DLFS1_NO_MALLOC -DLFS1_NO_DEBUG -DLFS1_NO_WARN -DLFS1_NO_ERROR -DLFS1_NO_ASSERT -DLFS2_NO_MALLOC -DLFS2_NO_DEBUG -DLFS2_NO_WARN -DLFS2_NO_ERROR -DLFS2_NO_ASSERT -I../../lib/berkeley-db-1.xx/PORT/include -I. -I../.. -Ibuild-standard -I../../shared/readline -Wall -Werror -Wextra -Wno-unused-parameter -Wpointer-arith -Wdouble-promotion -Wfloat-conversion -std=gnu99 -DUNIX -Os -DNDEBUG -fdata-sections -ffunction-sections -Ivariants/standard  -g -U _FORTIFY_SOURCE -DMICROPY_USE_READLINE=1 -DMICROPY_PY_TERMIOS=1 -DMICROPY_PY_SOCKET=1 -DMICROPY_PY_THREAD=1 -DMICROPY_PY_THREAD_GIL=0 -I/Library/Developer/CommandLineTools/SDKs/MacOSX12.sdk/usr/include/ffi -DMICROPY_PY_FFI=1 -DMPZ_DIG_SIZE=16  -DMICROPY_ROM_TEXT_COMPRESSION=1 -DMICROPY_QSTR_EXTRA_POOL=mp_qstr_frozen_const_pool -DMICROPY_MODULE_FROZEN_MPY -DMICROPY_MODULE_FROZEN_STR -c -MD -o build-standard/py/emitglue.o ../../py/emitglue.c || (echo -e "See \033[1;31mhttps://github.com/micropython/micropython/wiki/Build-Troubleshooting\033[0m"; false)
../../py/emitglue.c:125:9: error: operand must be a register in range [r0, r14] or apsr_nzcv
        "0:"
        ^
<inline asm>:1:16: note: instantiated into assembly here
        0:mrc p15, 0, r15, c7, c10, 3
                      ^
1 error generated.

I'm at git commit fd27770.
My system is a Mac Mini M1 (2020); uname -a reports:

Darwin Neds-mini 22.5.0 Darwin Kernel Version 22.5.0: Tue Apr 25 04:07:22 PDT 2023; root:xnu-8796.121.2~8/RELEASE_ARM64_T8103 arm64
@ned-pcs ned-pcs added the bug label Jun 11, 2023
@dlech
Copy link
Contributor
dlech commented Jun 11, 2023

Apple no longer allows running 32-bit apps on macOS so it doesn't make sense to enable MICROPY_FORCE_32BIT when targeting macOS.

@ned-pcs
Copy link
Contributor Author
ned-pcs commented Jun 11, 2023

Ah yes. Perhaps this should be removed from the Makefile?

@ned-pcs ned-pcs closed this as completed Jun 11, 2023
@dlech
Copy link
Contributor
dlech commented Jun 11, 2023

It is still used on Linux with x86_64 architecture.

@ned-pcs
Copy link
Contributor Author
ned-pcs commented Jun 12, 2023

I was wondering whether maybe it should be removed from the macOS part of the Makefile.

In ports/unix/Makefile, lines 98-103 contain:

ifeq ($(UNAME_S),Darwin)
ifeq ($(MICROPY_FORCE_32BIT),1)
CC = clang -m32
else
CC = clang
endif

@dpgeorge
Copy link
Member

It should still be possible to build in 32-bit mode on older macOS, right?

@ned-pcs
Copy link
Contributor Author
ned-pcs commented Jun 13, 2023

Pre-Catalina, yes.

@dpgeorge
Copy link
Member

Then I think the option to build in 32-bit mode should stay for now.

@ESP32DE
Copy link
ESP32DE commented Jun 30, 2023

Then I think the option to build in 32-bit mode should stay for now.

don't touch this 32 bit option :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@dlech @dpgeorge @ESP32DE @ned-pcs
0