[PATCH] D28990: Align i128 to 16 bytes

Eric Christopher via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 6 19:57:49 PDT 2017


echristo added a comment.

In https://reviews.llvm.org/D28990#801641, @vchuravy wrote:

> @echristo I noticed that you reverted r294702 shortly after it landed. What needs to be done to get it working?


From the revert:

commit 8c54b816e554ff9f9a75ec8a1d50198d2bfd9633
Author: Eric Christopher <echristo at gmail.com>
Date:   Fri Feb 10 04:35:32 2017 +0000

  Temporarily revert "For X86-64 linux and PPC64 linux align int128 to 16 bytes."
  until we can get better TargetMachine::isCompatibleDataLayout to compare - otherwise
  we can't code generate existing bitcode without a string equality data layout.
  
  This reverts commit r294702.
  
  git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@294709 91177308-0d34-0410-b5e6-96231b3b80d8


https://reviews.llvm.org/D28990





More information about the llvm-commits mailing list