[llvm] [llvm-objdump][BPF] --symbolize-operands: infer local labels for BPF (PR #100550)

via llvm-commits llvm-commits at lists.llvm.org
Sun Jul 28 21:50:19 PDT 2024


4ast wrote:

> > @yonghong-song , @4ast , do we want to make --symbolize-operands default for bpf and if so, should this be a part of this commit or a #95103? I understand why @MaskRay does not like it, but usability benefits seem significant.
> 
> The patch can land as is. The default --symbolize-operands does not need to be combined. It would be a pretty significant policy change as we do not allow targets to customize the defaults. That would require more debates.

No. User experience matters a lot more than some "policy".
We're going to keep L in symbol table then.

https://github.com/llvm/llvm-project/pull/100550


More information about the llvm-commits mailing list