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

Compilation flags for ARM64 on Mac #1

Open
wants to merge 1 commit into
base: legacy
Choose a base branch
from

Conversation

pfoof
Copy link

@pfoof pfoof commented Jul 23, 2022

Added compilation flags for ARM64

<file name="${NATIVE_TOOLKIT_PATH}/pixman/src/pixman-timer.c" />
<file name="${NATIVE_TOOLKIT_PATH}/pixman/src/pixman-trap.c" />
<file name="${NATIVE_TOOLKIT_PATH}/pixman/src/pixman-utils.c" />
<file name="${NATIVE_TOOLKIT_PATH}/pixman/src/pixman-x86.c" />
<file name="${NATIVE_TOOLKIT_PATH}/pixman/src/pixman-x86.c" if="windows || mac || linux" unless="rpi || HXCPP_ARM64"/>
Copy link
Member

@tobil4sk tobil4sk Mar 1, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not including pixman-x86.c and pixman-mips.c causes linking errors:

"__pixman_mips_get_implementations", referenced from:
    __pixman_choose_implementation in 2ec37e17_pixman-implementation.o
"__pixman_x86_get_implementations", referenced from:
    __pixman_choose_implementation in 2ec37e17_pixman-implementation.o

Both pixman-x86.c and pixman-mips.c should be built also on arm64 platforms, otherwise lime.ndll will not link properly. Instead, the config.h needs to be adjusted to fix compilation. See: #2

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for taking up this issue ^^

@tobil4sk tobil4sk mentioned this pull request Jul 5, 2024
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