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

Reply via email to