[PATCH] D123800: [BPF] handle opaque-pointer for __builtin_preserve_enum_value

Yonghong Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 14 09:08:15 PDT 2022


yonghong-song created this revision.
yonghong-song added reviewers: anakryiko, ast.
Herald added a subscriber: hiraditya.
Herald added a project: All.
yonghong-song requested review of this revision.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.

Opaque pointer [1] is enabled as the default with commit [2].
Andrii found that current __builtin_preserve_enum_value() can only handle
non opaque pointer code pattern and will segfault with latest
llvm main branch where opaque-pointer is enabled by default.

This patch added the opaque pointer support.
Besides llvm selftests, also verified with bpf-next bpf selftests.

  [1] https://llvm.org/docs/OpaquePointers.html
  [2] https://reviews.llvm.org/D123122


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D123800

Files:
  llvm/lib/Target/BPF/BPFAbstractMemberAccess.cpp
  llvm/test/CodeGen/BPF/CORE/intrinsic-typeinfo-enum-value-opaque-pointer.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D123800.422887.patch
Type: text/x-patch
Size: 6570 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220414/10884581/attachment.bin>


More information about the llvm-commits mailing list