[PATCH v2 1/1] objdump, as: Add callx support for BPF CPU v1
Jose E. Marchesi
jose.marchesi@oracle.com
Wed Feb 14 10:58:21 GMT 2024
> First, as I said yesterday in my direct message to you, thank you for
> making binutils such a pleasant place to contribute to FOSS. You have
> no idea what that means to contributors like me!
>
> Second, thank you for this helpful critique. I really appreciated
> reading your feedback and will reply inline below (including with an
> offer for a patch unrelated to callx that may clean up some of the
> non-constant uses throughout the bpf-specific code).
>
> Third, there is good news: The heavy lifting of this patch is largely
> "overcome by events" -- clang/llvm developers are changing their
> encoding of the callx instruction to more closely match what gcc does.
> In fact, v3 of this patch will look much more like v1 than v2.
Hi Will.
It seems to me that all we need binutils-wise is to enable the callx
instruction with BPF >= v1. No other changes are necessary as far as I
can see, other than adjusting the testsuite accordingly.
Thanks!
More information about the Binutils
mailing list