[PATCH] D28990: Align i128 to 16 bytes

Simonas Kazlauskas via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Jan 22 06:19:41 PST 2017


nagisa created this revision.
Herald added subscribers: nemanjai, mehdi_amini, jyknight.

This is what clang does as can be seen in [1]. As Clang does not support using 128-bit integers on
non-64 bit targets, no effort was spent on updating the data-layouts for 32-bit only backends.

This is wanted for rustc, which wants data-layout used by LLVM and data-layout used by rustc 
match to some extent.

[1]: https://github.com/llvm-mirror/clang/blob/916645c2637f8c80948eedcdea02258d0a6f79f5/lib/AST/ASTContext.cpp#L1734-L1738


https://reviews.llvm.org/D28990

Files:
  lib/Target/Mips/MipsTargetMachine.cpp
  lib/Target/PowerPC/PPCTargetMachine.cpp
  lib/Target/Sparc/SparcTargetMachine.cpp
  lib/Target/SystemZ/SystemZTargetMachine.cpp
  lib/Target/X86/X86TargetMachine.cpp
  test/CodeGen/Mips/llvm-ir/add.ll
  test/CodeGen/Mips/llvm-ir/and.ll
  test/CodeGen/Mips/llvm-ir/not.ll
  test/CodeGen/Mips/llvm-ir/or.ll
  test/CodeGen/Mips/llvm-ir/sub.ll
  test/CodeGen/Mips/llvm-ir/xor.ll
  test/CodeGen/X86/catchpad-dynamic-alloca.ll
  test/CodeGen/X86/libcall-sret.ll
  test/CodeGen/X86/mul-i1024.ll
  test/CodeGen/X86/mul-i512.ll
  test/CodeGen/X86/osx-private-labels.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D28990.85277.patch
Type: text/x-patch
Size: 28060 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170122/3aa75cac/attachment.bin>


More information about the llvm-commits mailing list