[PATCH] D130900: [SystemZ] Make 128 bit integers be aligned to 8 bytes.

[Jonas Paulsson via Phabricator via llvm-commits]

jonpa updated this revision to Diff 449301.
jonpa marked an inline comment as done.
jonpa added a comment.

Change to DataLayout string removed.

Test added to check that the middle end (and hopefully then the backend as well) will produce the right alignments when unspecified in the IR input.

@uweigand I also added tests for the alignment of a vector depending on the presence of the vector facility. I am not sure if I have run into something weird here, but I left the test and output this way for now so you can see what I mean: If I use -mattr=-vector the alignment becomes 16 for the vector, but not if this is specified in the function attributes only (see @f2).


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D130900/new/

https://reviews.llvm.org/D130900

Files:
  clang/include/clang/Basic/TargetInfo.h
  clang/lib/AST/ASTContext.cpp
  clang/lib/Basic/TargetInfo.cpp
  clang/lib/Basic/Targets/SystemZ.h
  clang/test/CodeGen/SystemZ/align-systemz.c
  clang/test/CodeGen/SystemZ/systemz-abi.c
  clang/test/CodeGen/SystemZ/zos-alignment.c
  llvm/test/CodeGen/SystemZ/unaligned-02.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D130900.449301.patch
Type: text/x-patch
Size: 6207 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220802/b1a10d25/attachment.bin>