[all-commits] [llvm/llvm-project] c48b46: [BPF] Don't fetch alignment of llvm.preserve.union...

Nikita Popov via All-commits all-commits at lists.llvm.org
Fri Mar 25 01:53:47 PDT 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: c48b4641c7baf9f7280d352c13299b3aa0203179
      https://github.com/llvm/llvm-project/commit/c48b4641c7baf9f7280d352c13299b3aa0203179
  Author: Nikita Popov <npopov at redhat.com>
  Date:   2022-03-25 (Fri, 25 Mar 2022)

  Changed paths:
    M llvm/lib/Target/BPF/BPFAbstractMemberAccess.cpp

  Log Message:
  -----------
  [BPF] Don't fetch alignment of llvm.preserve.union.access.index argument

The way the alignment is determined here is not compatible with
opaque pointers -- we'd have to thread it through using either
align or elementtype attributes.

However, as far as I can tell this alignment is actually never
used for this particular intrinsic, so I've dropped the assignment
entirely and converted RecordAlignment to MaybeAlign, so we get an
assertion failure if it does end up being used.

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




More information about the All-commits mailing list