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

Jonas Paulsson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 2 09:12:58 PDT 2022


jonpa added inline comments.


================
Comment at: clang/lib/Basic/Targets/SystemZ.h:60
     } else
-      resetDataLayout("E-m:e-i1:8:16-i8:8:16-i64:64-f128:64-a:8:16-n32:64");
+      resetDataLayout("E-m:e-i1:8:16-i8:8:16-i64:64-i128:64-f128:64-a:8:16-n32:64");
     MaxAtomicPromoteWidth = MaxAtomicInlineWidth = 64;
----------------
nikic wrote:
> Isn't this the default? We should be using the alignment of the largest specified integer type, which is 64.
You are right - removed.


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

https://reviews.llvm.org/D130900



More information about the llvm-commits mailing list