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

PowerPC C ZST ABI fixes #64259

Closed
wants to merge 1 commit into from
Closed

Commits on Nov 11, 2019

  1. Fix zero-sized aggregate ABI on powerpc

    For targets that pass zero-sized aggregates indirectly (generally
    those that pass all aggregates indirectly), we must allocate a register
    for passing the address of the ZST. Clean up the existing cases and add
    powerpc, which requires this as well.
    
    While there are not currently musl targets for s390x or sparc64, they
    would have the same ABI as gnu targets, so remove the env == "gnu" check
    in the Linux case.
    
    Ideally, since it is a property of the C ABI, the `!rust_abi` case would
    be handled entirely in `adjust_c_abi`. However, that would require
    updating each implementation of `compute_abi_info` to handle ZSTs.
    smaeul committed Nov 11, 2019
    Configuration menu
    Copy the full SHA
    308071a View commit details
    Browse the repository at this point in the history