]> granicus.if.org Git - clang/commit
bpf: accept all asm register names
authorYonghong Song <yhs@fb.com>
Wed, 11 Apr 2018 16:08:00 +0000 (16:08 +0000)
committerYonghong Song <yhs@fb.com>
Wed, 11 Apr 2018 16:08:00 +0000 (16:08 +0000)
commita5507dea800fe8cd338139432a7438c33e7027f3
treef34f954861cdd1dc0a3bc2fe6f05dfe214949b17
parent7c0049b35f52c6ae7d2cf945688abd1a799ef4e7
bpf: accept all asm register names

Sometimes when people compile bpf programs with
"clang ... -target bpf ...", the kernel header
files may contain host arch inline assembly codes
as in the patch https://patchwork.kernel.org/patch/10119683/
by Arnaldo Carvaldo de Melo.

The current workaround in the above patch
is to guard the inline assembly with "#ifndef __BPF__"
marco. So when __BPF__ is defined, these macros will
have no use.

Such a method is not extensible. As a matter of fact,
most of these inline assembly codes will be thrown away
at the end of clang compilation.

So for bpf target, this patch accepts all asm register
names in clang AST stage. The name will be checked
again during llc code generation if the inline assembly
code is indeed for bpf programs.

With this patch, the above "#ifndef __BPF__" is not needed
any more in https://patchwork.kernel.org/patch/10119683/.

Signed-off-by: Yonghong Song <yhs@fb.com>
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@329823 91177308-0d34-0410-b5e6-96231b3b80d8
include/clang/Basic/TargetInfo.h
lib/Basic/Targets/BPF.h