Skip to content

Don't define CTYPES_PASS_BY_REF_HACK on aarch64 or riscv64 #103438

Open

Description

@andreas-schwab

There is no need to define CTYPES_PASS_BY_REF_HACK on aarch64 or riscv64. This was a bug in libffi that has since been been fixed, for riscv64 in commit 4b0c358 and for aarch64 in commit e409225.

Linked PRs

Activity

  1. FFY00 commented on Apr 11, 2023

    @FFY00
    Member

    The fix is only available on libfii 3.4.4, for riscv, and 3.4.3, for aarch, so it'd make sense to document this.

  2. ronaldoussoren commented on Apr 21, 2023

    @ronaldoussoren
    Contributor

    Unconditionally removing the workaround for aarch64 could break builds on macOS that use the system install of libffi, including the installers on our website. Therefore I'm -1 on unconditionally removing the workaround.

    Is the need for using the workaround something that can be tested in the configure script?

    Even then it would be preferable to continue using the workaround on macOS because we also target older macOS versions that likely will never see this bug fix in the system install of libffi. Using the system install of libffi is desirable due to code signing "tricks" in that build of libffi.

  3. assigned and unassigned on Apr 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    Projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions