[PATCH] D89083: [BPF] Make BPFAbstractMemberAccessPass required

Yonghong Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 9 11:00:22 PDT 2020


yonghong-song added a comment.

Thanks for explanation for IsRequired(), really appreciated.

Could you change optnone-2.ll with the following source:
-bash-4.4$ cat t.c
struct ss { int a; };
int foo() { return __builtin_btf_type_id(0, 0) + __builtin_preserve_type_info(*(struct ss *)0, 0); }
-bash-4.4$

This will cover both BPFAbstractMemberAccess.cpp and BPFPreserveDIType.cpp. This way, we do not need store-addr-optnone.ll.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D89083



More information about the llvm-commits mailing list