[all-commits] [llvm/llvm-project] 967eeb: [AIX] Pack BasicBlockBits

David Tenty via All-commits all-commits at lists.llvm.org
Thu Feb 20 11:28:52 PST 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 967eeb109bedc4ae606fdf6ad6eca58ffbac6739
      https://github.com/llvm/llvm-project/commit/967eeb109bedc4ae606fdf6ad6eca58ffbac6739
  Author: David Tenty <daltenty at ibm.com>
  Date:   2020-02-20 (Thu, 20 Feb 2020)

  Changed paths:
    M llvm/include/llvm/IR/BasicBlock.h

  Log Message:
  -----------
  [AIX] Pack BasicBlockBits

Summary:
D51664 introduces a new structure BasicBlockBits which it expects to be
packed a certain way. This change is very similar to D60164, and we apply the
same fix:

"On AIX, the canonical layout of bit-fields would cause
these ... to span four bytes. Applying the pack pragma for compilers that
employ the AIX canonical layout allows these ... to fit within the expected
two bytes. In the future, the pragma would also likely need to be applied when
building with Clang on AIX."

Reviewers: jasonliu, hubert.reinterpretcast, sfertile, xingxue

Reviewed By: sfertile

Subscribers: llvm-commits

Tags: #llvm

Differential Revision: https://reviews.llvm.org/D74911




More information about the All-commits mailing list