[llvm] [BPF] Use ".L" local prefix label for basic blocks (PR #95103)
via llvm-commits
llvm-commits at lists.llvm.org
Sat Jul 27 17:12:28 PDT 2024
yonghong-song wrote:
@4ast For this ' I was hoping that switch to ".L" will make labels to appear in disasm automatically without extra flag.', the switch to '.L' won't change anything to disasm since with '.L' the labels still not in symbol table.
https://github.com/llvm/llvm-project/pull/100550 is the right direction as at least with option --symbolize-operands, the disasm will break basic blocks with labels properly. Without https://github.com/llvm/llvm-project/pull/100550, even with --symbolize-operands, the disasm will display without any labels.
But at you mentioned, this is a regression. Previous, 'llvm-objdump -d ...' will be able to disasm with proper labels. Now we need 'llvm-objdump --symbolize-operands -d ...'. That is why I suggest to make '--symbolize-operands' as the default at least for BPF. An option '--no-symbolize-operands' can be added to disable labels.
https://github.com/llvm/llvm-project/pull/95103
More information about the llvm-commits
mailing list