yonghong-song added a comment. In D81479#2083510 <https://reviews.llvm.org/D81479#2083510>, @anakryiko wrote:
> IntrinsicsBPF.td uses `__builtin_bpf_load_u32_to_ptr`, while everywhere it is > just `__builtin_load_u32_to_ptr`. I think having bpf prefix there is good, > given this is bpf-specific. But either way, probably should be consistent > everywhere? Sounds good. This is indeed specific to bpf, adding bpf prefix to the public builtin name sounds reasonable. Will make the change. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D81479/new/ https://reviews.llvm.org/D81479 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits