[PATCH] D61173: [BPF] do not generate predefined macro bpf
Yonghong Song via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Apr 25 23:09:23 PDT 2019
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
More information about the cfe-commits
mailing list