Messages in this thread | | | Date | Thu, 18 May 2023 12:05:53 +0800 | From | Leo Yan <> | Subject | Re: [PATCH] perf symbol: Add LoongArch case in get_plt_sizes() |
| |
On Thu, May 18, 2023 at 11:57:29AM +0800, Huacai Chen wrote: > On Thu, May 18, 2023 at 11:21 AM Leo Yan <leo.yan@linaro.org> wrote: > > > > On Thu, May 18, 2023 at 11:12:26AM +0800, Huacai Chen wrote: > > > On Thu, May 18, 2023 at 11:06 AM Leo Yan <leo.yan@linaro.org> wrote: > > > > > > > > On Thu, May 18, 2023 at 10:11:27AM +0800, Huacai Chen wrote: > > > > > Queued, thanks. > > > > > > > > The patch is fine for me. > > > > > > > > Should not perf patches are to be merged via Arnaldo's tree? > > > > > > I think both are OK, if Arnaldo takes this patch, I will drop it. > > > > A good practice is to firstly inquiry the maintainers. > > > > AFAIK, Arnaldo will test perf patches before sending out pull request; > > if perf patches are scattered out, it might be out of the testing > > radar. > OK, I know, thank you very much.
You are welcome!
I found the code base for bfd: https://github.com/bminor/binutils-gdb/blob/master/bfd/elfnn-loongarch.c
And this patch is consistent with above link, FWIW:
Reviewed-by: Leo Yan <leo.yan@linaro.org>
| |