Intrinsic has incorrect return type. LLVM ERROR: Broken function found, compilation aborted. aarch64 / ARM / nightly #118124

Activity

1. spital

   added

   C-bugCategory: This is a bug.Category: This is a bug.

   on Nov 21, 2023
2. rustbot

   added

   needs-triageThis issue may need triage. Remove when done. See docs forge.rust-lang.org/release/issue-triagingThis issue may need triage. Remove when done. See docs forge.rust-lang.org/release/issue-triaging

   on Nov 21, 2023
3. Noratrieb

   added

   T-compilerRelevant to the compiler team, which will review and decide on the PR/issue.Relevant to the compiler team, which will review and decide on the PR/issue.

   E-needs-bisectionCall for participation: This issue needs bisection: https://github.com/rust-lang/cargo-bisect-rustcCall for participation: This issue needs bisection: https://github.com/rust-lang/cargo-bisect-rustc

   O-AArch64Armv8-A or later processors in AArch64 modeArmv8-A or later processors in AArch64 mode

   and removed

   needs-triageThis issue may need triage. Remove when done. See docs forge.rust-lang.org/release/issue-triagingThis issue may need triage. Remove when done. See docs forge.rust-lang.org/release/issue-triaging

   on Nov 21, 2023
4. nikic

   added

   A-LLVMArea: Code generation parts specific to LLVM. Both correctness bugs and optimization-related issues.Area: Code generation parts specific to LLVM. Both correctness bugs and optimization-related issues.

   on Nov 21, 2023
5. Noratrieb commented on Nov 21, 2023

   

   Noratrieb

   on Nov 21, 2023

   Member

   More actions

   searched nightlies: from nightly-2023-11-19 to nightly-2023-11-20
   regressed nightly: nightly-2023-11-20
   searched commit range: 4cb3bee...9a66e44
   regressed commit: d19980e

   bisected with cargo-bisect-rustc v0.6.5

   Host triple: aarch64-unknown-linux-gnu
   Reproduce with:

   cargo bisect-rustc --start 2023-11-19 --access github --end 2023-11-20 --regress error

   looks like something in that PR changed ABI behavior and now LLVM and rustc disagree. Don't know who's wrong. @RalfJung @davidtwco
6. Noratrieb

   added

   regression-from-stable-to-nightlyPerformance or correctness regression from stable to nightly.Performance or correctness regression from stable to nightly.

   and removed

   E-needs-bisectionCall for participation: This issue needs bisection: https://github.com/rust-lang/cargo-bisect-rustcCall for participation: This issue needs bisection: https://github.com/rust-lang/cargo-bisect-rustc

   on Nov 21, 2023
7. rustbot

   added

   I-prioritizeIssue needs a team member to assess the impact. Will be replaced by P-{low,medium,high,critical}Issue needs a team member to assess the impact. Will be replaced by P-{low,medium,high,critical}

   on Nov 21, 2023
8. RalfJung commented on Nov 21, 2023

   

   RalfJung

   on Nov 21, 2023

   Last edited by RalfJung

   Member

   More actions

   Hm, I don't know how that "unadjusted" ABI stuff works...

   pub unsafe fn vld1q_s8_x4(a: *const i8) -> int8x16x4_t {
       #[allow(improper_ctypes)]
       extern "unadjusted" {
           #[cfg_attr(target_arch = "arm", link_name = "llvm.arm.neon.vld1x4.v16i8.p0i8")]
           #[cfg_attr(target_arch = "aarch64", link_name = "llvm.aarch64.neon.ld1x4.v16i8.p0i8")]
           fn vld1q_s8_x4_(a: *const i8) -> int8x16x4_t;
       }
   vld1q_s8_x4_(a)
   }
9. RalfJung commented on Nov 21, 2023

   

   RalfJung

   on Nov 21, 2023

   Member

   More actions

   #118127 should fix this. I am not sure how to add a test.
10. apiraino commented on Nov 23, 2023

    

    apiraino

    on Nov 23, 2023

    Contributor

    More actions

    WG-prioritization assigning priority (Zulip discussion).

    @rustbot label -I-prioritize +P-high
11. rustbot

    added

    P-highHigh priorityHigh priority

    and removed

    I-prioritizeIssue needs a team member to assess the impact. Will be replaced by P-{low,medium,high,critical}Issue needs a team member to assess the impact. Will be replaced by P-{low,medium,high,critical}

    on Nov 23, 2023
12. bors

    added a commit that references this issue on Nov 25, 2023

    Auto merge of rust-lang#118127 - RalfJung:unadjusted-abi, r=compiler-…

    ...

    16087ee
13. bors

    closed this as completedin 16087eeon Nov 25, 2023
14. github-actions

    added a commit that references this issue on Nov 27, 2023

    Auto merge of #118127 - RalfJung:unadjusted-abi, r=compiler-errors

    ...

    1e0df89