[SPARC] Align i128 to 16 bytes in SPARC datalayouts by koachan · Pull Request #106951 · llvm/llvm-project

@koachan

@koachan

nikic

@koachan

nikic

@koachan

@koachan

s-barannikov

@rorth rorth mentioned this pull request

Sep 30, 2024

koachan added a commit that referenced this pull request

Oct 2, 2024
It turns out that we cannot rely on the presence of `-i64:64` as a
position reference when adding the `-i128:128` datalayout string due to
some custom datalayout strings lacking it (e.g ones used by bugpoint,
among other things).
Do not add the `-i128:128` string in that case.

This fixes the regression introduced in
#106951.

maurer added a commit to maurer/rust that referenced this pull request

Oct 31, 2024
LLVM continues to align more 128-bit integers to 128-bits in the data
layout rather than relying on the high level language to do it. Update
SPARC target files to match and add a backcompat replacement for current
LLVMs.

See llvm/llvm-project#106951 for details

workingjubilee added a commit to workingjubilee/rustc that referenced this pull request

Nov 1, 2024
llvm: Match new LLVM 128-bit integer alignment on sparc

LLVM continues to align more 128-bit integers to 128-bits in the data layout rather than relying on the high level language to do it. Update SPARC target files to match and add a backcompat replacement for current LLVMs.

See llvm/llvm-project#106951 for details

`@rustbot` label: +llvm-main

r? `@durin42`

(Please wait for the LLVM CI to come back before approving), creating this PR to get it tested there.

rust-timer added a commit to rust-lang-ci/rust that referenced this pull request

Nov 1, 2024
Rollup merge of rust-lang#132422 - maurer:sparc-layout, r=durin42

llvm: Match new LLVM 128-bit integer alignment on sparc

LLVM continues to align more 128-bit integers to 128-bits in the data layout rather than relying on the high level language to do it. Update SPARC target files to match and add a backcompat replacement for current LLVMs.

See llvm/llvm-project#106951 for details

`@rustbot` label: +llvm-main

r? `@durin42`

(Please wait for the LLVM CI to come back before approving), creating this PR to get it tested there.

alexrp