yonghong-song marked an inline comment as done. yonghong-song added inline comments.
================ Comment at: lib/Basic/Targets/BPF.cpp:23 MacroBuilder &Builder) const { - DefineStd(Builder, "bpf", Opts); + Builder.defineMacro("__bpf"); + Builder.defineMacro("__bpf__"); ---------------- ast wrote: > I don't think anyone is using this one. > May be we can keep ` __bpf__ ` and ` __BPF__ ` only? We can do this for sure. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D61173/new/ https://reviews.llvm.org/D61173 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits